Skip to content

Commit

Permalink
Merge pull request #5430 from Expensify/amal-query-param-transition
Browse files Browse the repository at this point in the history
Rename route so that we expect traditional query paramters

(cherry picked from commit 1c6c70f)
  • Loading branch information
Amal Nazeem authored and OSBotify committed Sep 23, 2021
1 parent e93f1cf commit 7def926
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/ROUTES.js
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ export default {
getReportDetailsRoute: reportID => `r/${reportID}/details`,
VALIDATE_LOGIN: 'v',
VALIDATE_LOGIN_WITH_VALIDATE_CODE: 'v/:accountID/:validateCode',
LOGIN_WITH_SHORT_LIVED_TOKEN: 'transition/:accountID/:email/:shortLivedToken/:encryptedAuthToken/:exitTo',
LOGIN_WITH_SHORT_LIVED_TOKEN: 'transition',

// This is a special validation URL that will take the user to /workspace/new after validation. This is used
// when linking users from e.com in order to share a session in this app.
Expand Down
2 changes: 1 addition & 1 deletion src/libs/Navigation/NavigationRoot.js
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ class NavigationRoot extends Component {
const path = getPathFromState(state, linkingConfig.config);

// Don't log the route transitions from OldDot because they contain authTokens
if (path.includes('/transition/')) {
if (path.includes('/transition')) {
Log.info('Navigating from transition link from OldDot using short lived authToken');
} else {
Log.info('Navigating to route', false, {path});
Expand Down

0 comments on commit 7def926

Please sign in to comment.