return early when we've already initiated saml login

Expensify · Nov 17, 2023 · 79e89c2 · 79e89c2
1 parent 6b5236d
commit 79e89c2
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/src/pages/signin/SAMLSignInPage/index.native.js b/src/pages/signin/SAMLSignInPage/index.native.js
@@ -48,7 +48,7 @@ function SAMLSignInPage({credentials}) {
             }
 
             // If the login attempt is unsuccessful, set the error message for the account and redirect to sign in page
-            if (searchParams.has('error')) {;
+            if (searchParams.has('error')) {
                 Session.setAccountError(searchParams.get('error'));
                 Navigation.navigate(ROUTES.HOME);
             }

diff --git a/src/pages/signin/SignInPage.js b/src/pages/signin/SignInPage.js
@@ -192,6 +192,9 @@ function SignInPage({credentials, account, isInModal, activeClients, preferredLo
     if (shouldInitiateSAMLLogin) {
         setHasInitiatedSAMLLogin(true);
         Navigation.isNavigationReady().then(() => Navigation.navigate(ROUTES.SAML_SIGN_IN));
+    } else if (hasInitiatedSAMLLogin) {
+        // Return early because we're already navigating to a different page
+        return;
     }
 
     let welcomeHeader = '';