Merge pull request #15262 from allroundexperts/fix-15059

Pass down url parameters when the native app is opened from browser
Expensify · Feb 28, 2023 · 411dfc9 · 411dfc9
2 parents 3ea9e74 + 151833d
commit 411dfc9
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/desktop/main.js b/desktop/main.js
@@ -246,7 +246,7 @@ const mainWindow = (() => {
         app.on('open-url', (event, url) => {
             event.preventDefault();
             const urlObject = new URL(url);
-            deeplinkUrl = `${APP_DOMAIN}${urlObject.pathname}`;
+            deeplinkUrl = `${APP_DOMAIN}${urlObject.pathname}${urlObject.search}${urlObject.hash}`;
 
             if (browserWindow) {
                 browserWindow.loadURL(deeplinkUrl);

diff --git a/src/components/DeeplinkWrapper/index.website.js b/src/components/DeeplinkWrapper/index.website.js
@@ -74,7 +74,7 @@ class DeeplinkWrapper extends PureComponent {
 
     openRouteInDesktopApp() {
         const expensifyUrl = new URL(CONFIG.EXPENSIFY.NEW_EXPENSIFY_URL);
-        const expensifyDeeplinkUrl = `${CONST.DEEPLINK_BASE_URL}${expensifyUrl.host}${window.location.pathname}`;
+        const expensifyDeeplinkUrl = `${CONST.DEEPLINK_BASE_URL}${expensifyUrl.host}${window.location.pathname}${window.location.search}${window.location.hash}`;
 
         // This check is necessary for Safari, otherwise, if the user
         // does NOT have the Expensify desktop app installed, it's gonna