From 71b2bb91cfc3ccf56ea2274637373a527bb8ecf4 Mon Sep 17 00:00:00 2001 From: Bernhard Owen Josephus Date: Fri, 12 Apr 2024 22:43:13 +0800 Subject: [PATCH 1/2] fix here is your magic code shows briefly --- src/pages/ValidateLoginPage/index.website.tsx | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/src/pages/ValidateLoginPage/index.website.tsx b/src/pages/ValidateLoginPage/index.website.tsx index b8e8709215e8..9fb97aa46529 100644 --- a/src/pages/ValidateLoginPage/index.website.tsx +++ b/src/pages/ValidateLoginPage/index.website.tsx @@ -24,9 +24,11 @@ function ValidateLoginPage({ const isSignedIn = !!session?.authToken && session?.authTokenType !== CONST.AUTH_TOKEN_TYPES.ANONYMOUS; const is2FARequired = !!account?.requiresTwoFactorAuth; const cachedAccountID = credentials?.accountID; + const isUserClickedSignIn = !login && isSignedIn && (autoAuthState === CONST.AUTO_AUTH_STATE.SIGNING_IN || autoAuthState === CONST.AUTO_AUTH_STATE.JUST_SIGNED_IN); + const shouldStartSignInWithValidateCode = !isUserClickedSignIn && !isSignedIn && (login || exitTo); useEffect(() => { - if (!login && isSignedIn && (autoAuthState === CONST.AUTO_AUTH_STATE.SIGNING_IN || autoAuthState === CONST.AUTO_AUTH_STATE.JUST_SIGNED_IN)) { + if (isUserClickedSignIn) { // The user clicked the option to sign in the current tab Navigation.isNavigationReady().then(() => { Navigation.goBack(); @@ -35,7 +37,7 @@ function ValidateLoginPage({ } Session.initAutoAuthState(autoAuthState); - if (isSignedIn || (!login && !exitTo)) { + if (!shouldStartSignInWithValidateCode) { if (exitTo) { Session.handleExitToNavigation(exitTo); } @@ -71,13 +73,13 @@ function ValidateLoginPage({ {autoAuthState === CONST.AUTO_AUTH_STATE.FAILED && } {autoAuthState === CONST.AUTO_AUTH_STATE.JUST_SIGNED_IN && is2FARequired && !isSignedIn && } {autoAuthState === CONST.AUTO_AUTH_STATE.JUST_SIGNED_IN && isSignedIn && !exitTo && } - {autoAuthState === CONST.AUTO_AUTH_STATE.NOT_STARTED && !exitTo && ( + {autoAuthState === CONST.AUTO_AUTH_STATE.NOT_STARTED && !exitTo && !shouldStartSignInWithValidateCode && ( )} - {autoAuthState === CONST.AUTO_AUTH_STATE.SIGNING_IN && } + {(autoAuthState === CONST.AUTO_AUTH_STATE.SIGNING_IN || shouldStartSignInWithValidateCode) && } ); } From bdeeb5a908c8ad0ab74fcc0b1718acdd8c327eed Mon Sep 17 00:00:00 2001 From: Bernhard Owen Josephus Date: Fri, 12 Apr 2024 23:00:50 +0800 Subject: [PATCH 2/2] lint --- src/pages/ValidateLoginPage/index.website.tsx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/pages/ValidateLoginPage/index.website.tsx b/src/pages/ValidateLoginPage/index.website.tsx index 9fb97aa46529..035ad1492f01 100644 --- a/src/pages/ValidateLoginPage/index.website.tsx +++ b/src/pages/ValidateLoginPage/index.website.tsx @@ -25,7 +25,7 @@ function ValidateLoginPage({ const is2FARequired = !!account?.requiresTwoFactorAuth; const cachedAccountID = credentials?.accountID; const isUserClickedSignIn = !login && isSignedIn && (autoAuthState === CONST.AUTO_AUTH_STATE.SIGNING_IN || autoAuthState === CONST.AUTO_AUTH_STATE.JUST_SIGNED_IN); - const shouldStartSignInWithValidateCode = !isUserClickedSignIn && !isSignedIn && (login || exitTo); + const shouldStartSignInWithValidateCode = !isUserClickedSignIn && !isSignedIn && (!!login || !!exitTo); useEffect(() => { if (isUserClickedSignIn) {