Skip to content

Commit 767367b

Browse files
authored
Merge pull request #40760 from Expensify/revert-40177-fix/39904-here-is-ur-magic-code-shows-briefly
[CP Staging] Revert "Fix here is your magic code shows briefly when login with magic link"
2 parents f05d306 + 41ee713 commit 767367b

File tree

1 file changed

+4
-6
lines changed

1 file changed

+4
-6
lines changed

src/pages/ValidateLoginPage/index.website.tsx

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -24,11 +24,9 @@ function ValidateLoginPage({
2424
const isSignedIn = !!session?.authToken && session?.authTokenType !== CONST.AUTH_TOKEN_TYPES.ANONYMOUS;
2525
const is2FARequired = !!account?.requiresTwoFactorAuth;
2626
const cachedAccountID = credentials?.accountID;
27-
const isUserClickedSignIn = !login && isSignedIn && (autoAuthState === CONST.AUTO_AUTH_STATE.SIGNING_IN || autoAuthState === CONST.AUTO_AUTH_STATE.JUST_SIGNED_IN);
28-
const shouldStartSignInWithValidateCode = !isUserClickedSignIn && !isSignedIn && (!!login || !!exitTo);
2927

3028
useEffect(() => {
31-
if (isUserClickedSignIn) {
29+
if (!login && isSignedIn && (autoAuthState === CONST.AUTO_AUTH_STATE.SIGNING_IN || autoAuthState === CONST.AUTO_AUTH_STATE.JUST_SIGNED_IN)) {
3230
// The user clicked the option to sign in the current tab
3331
Navigation.isNavigationReady().then(() => {
3432
Navigation.goBack();
@@ -37,7 +35,7 @@ function ValidateLoginPage({
3735
}
3836
Session.initAutoAuthState(autoAuthState);
3937

40-
if (!shouldStartSignInWithValidateCode) {
38+
if (isSignedIn || (!login && !exitTo)) {
4139
if (exitTo) {
4240
Session.handleExitToNavigation(exitTo);
4341
}
@@ -73,13 +71,13 @@ function ValidateLoginPage({
7371
{autoAuthState === CONST.AUTO_AUTH_STATE.FAILED && <ExpiredValidateCodeModal />}
7472
{autoAuthState === CONST.AUTO_AUTH_STATE.JUST_SIGNED_IN && is2FARequired && !isSignedIn && <JustSignedInModal is2FARequired />}
7573
{autoAuthState === CONST.AUTO_AUTH_STATE.JUST_SIGNED_IN && isSignedIn && !exitTo && <JustSignedInModal is2FARequired={false} />}
76-
{autoAuthState === CONST.AUTO_AUTH_STATE.NOT_STARTED && !exitTo && !shouldStartSignInWithValidateCode && (
74+
{autoAuthState === CONST.AUTO_AUTH_STATE.NOT_STARTED && !exitTo && (
7775
<ValidateCodeModal
7876
accountID={Number(accountID)}
7977
code={validateCode}
8078
/>
8179
)}
82-
{(autoAuthState === CONST.AUTO_AUTH_STATE.SIGNING_IN || shouldStartSignInWithValidateCode) && <FullScreenLoadingIndicator />}
80+
{autoAuthState === CONST.AUTO_AUTH_STATE.SIGNING_IN && <FullScreenLoadingIndicator />}
8381
</>
8482
);
8583
}

0 commit comments

Comments
 (0)