Skip to content

Commit

Permalink
Merge pull request Expensify#52809 from bernhardoj/fix/52398-going-ba…
Browse files Browse the repository at this point in the history
…ck-doesn't-go-back-to-confirm-page

Fix going back from report detail doesn't bring to confirm page
  • Loading branch information
nkuoch authored Nov 25, 2024
2 parents f4f8da7 + 1633b7e commit 81724d9
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions src/components/MoneyRequestConfirmationList.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -706,14 +706,14 @@ function MoneyRequestConfirmationList({
const activeRoute = Navigation.getActiveRoute();

if (option.isSelfDM) {
Navigation.navigate(ROUTES.PROFILE.getRoute(currentUserPersonalDetails.accountID, activeRoute));
Navigation.navigate(ROUTES.PROFILE.getRoute(currentUserPersonalDetails.accountID, activeRoute), CONST.NAVIGATION.ACTION_TYPE.PUSH);
return;
}

if (option.accountID) {
Navigation.navigate(ROUTES.PROFILE.getRoute(option.accountID, activeRoute));
Navigation.navigate(ROUTES.PROFILE.getRoute(option.accountID, activeRoute), CONST.NAVIGATION.ACTION_TYPE.PUSH);
} else if (option.reportID) {
Navigation.navigate(ROUTES.REPORT_WITH_ID_DETAILS.getRoute(option.reportID, activeRoute));
Navigation.navigate(ROUTES.REPORT_WITH_ID_DETAILS.getRoute(option.reportID, activeRoute), CONST.NAVIGATION.ACTION_TYPE.PUSH);
}
};

Expand Down

0 comments on commit 81724d9

Please sign in to comment.