From 47354c8caea609a07db6d6a61a379fc9496fde7c Mon Sep 17 00:00:00 2001
From: Carlos Martins <cmartins@expensify.com>
Date: Tue, 12 Dec 2023 14:06:35 -0700
Subject: [PATCH] Merge pull request #32945 from dukenv0307/fix/32795

Not show prompt open in desktop app for magic link

(cherry picked from commit d376603d456f1023b7ea738fb6c3e78842eb17e1)
---
 src/Expensify.js                                | 6 +++++-
 src/components/DeeplinkWrapper/index.website.js | 8 +++++---
 2 files changed, 10 insertions(+), 4 deletions(-)

diff --git a/src/Expensify.js b/src/Expensify.js
index aece93c0ff4d..756df5b79b88 100644
--- a/src/Expensify.js
+++ b/src/Expensify.js
@@ -112,6 +112,7 @@ function Expensify(props) {
     }, [props.isCheckingPublicRoom]);
 
     const isAuthenticated = useMemo(() => Boolean(lodashGet(props.session, 'authToken', null)), [props.session]);
+    const autoAuthState = useMemo(() => lodashGet(props.session, 'autoAuthState', ''), [props.session]);
 
     const contextValue = useMemo(
         () => ({
@@ -207,7 +208,10 @@ function Expensify(props) {
     }
 
     return (
-        <DeeplinkWrapper isAuthenticated={isAuthenticated}>
+        <DeeplinkWrapper
+            isAuthenticated={isAuthenticated}
+            autoAuthState={autoAuthState}
+        >
             {shouldInit && (
                 <>
                     <GrowlNotification ref={Growl.growlRef} />
diff --git a/src/components/DeeplinkWrapper/index.website.js b/src/components/DeeplinkWrapper/index.website.js
index 166818e4ae27..d81c99657dd8 100644
--- a/src/components/DeeplinkWrapper/index.website.js
+++ b/src/components/DeeplinkWrapper/index.website.js
@@ -16,6 +16,8 @@ const propTypes = {
     children: PropTypes.node.isRequired,
     /** User authentication status */
     isAuthenticated: PropTypes.bool.isRequired,
+    /** The auto authentication status */
+    autoAuthState: PropTypes.string,
 };
 
 function isMacOSWeb() {
@@ -36,7 +38,7 @@ function promptToOpenInDesktopApp() {
         App.beginDeepLinkRedirect(!isMagicLink);
     }
 }
-function DeeplinkWrapper({children, isAuthenticated}) {
+function DeeplinkWrapper({children, isAuthenticated, autoAuthState}) {
     const [currentScreen, setCurrentScreen] = useState();
     const [hasShownPrompt, setHasShownPrompt] = useState(false);
     const removeListener = useRef();
@@ -69,7 +71,7 @@ function DeeplinkWrapper({children, isAuthenticated}) {
             return routeRegex.test(window.location.pathname);
         });
         // Making a few checks to exit early before checking authentication status
-        if (!isMacOSWeb() || isUnsupportedDeeplinkRoute || CONFIG.ENVIRONMENT === CONST.ENVIRONMENT.DEV || hasShownPrompt) {
+        if (!isMacOSWeb() || isUnsupportedDeeplinkRoute || hasShownPrompt || CONFIG.ENVIRONMENT === CONST.ENVIRONMENT.DEV || autoAuthState === CONST.AUTO_AUTH_STATE.NOT_STARTED) {
             return;
         }
         // We want to show the prompt immediately if the user is already authenticated.
@@ -92,7 +94,7 @@ function DeeplinkWrapper({children, isAuthenticated}) {
             promptToOpenInDesktopApp();
             setHasShownPrompt(true);
         }
-    }, [currentScreen, hasShownPrompt, isAuthenticated]);
+    }, [currentScreen, hasShownPrompt, isAuthenticated, autoAuthState]);
 
     return children;
 }