Merge pull request #26710 from Prince-Mendiratta/prince/fix-24291-man…

…datory-fallback [Navigation] Make fallback route necessary for Navigation.goBack method
Expensify · Sep 20, 2023 · 52dc94d · 52dc94d
2 parents b773455 + c07a584
commit 52dc94d
Show file tree

Hide file tree

Showing 28 changed files with 38 additions and 64 deletions.
diff --git a/src/components/BlockingViews/FullPageNotFoundView.js b/src/components/BlockingViews/FullPageNotFoundView.js
@@ -8,6 +8,7 @@ import Navigation from '../../libs/Navigation/Navigation';
 import variables from '../../styles/variables';
 import styles from '../../styles/styles';
 import useLocalize from '../../hooks/useLocalize';
+import ROUTES from '../../ROUTES';
 
 const propTypes = {
  /** Child elements */
@@ -44,7 +45,7 @@ const defaultProps = {
  titleKey: 'notFound.notHere',
  subtitleKey: 'notFound.pageNotFound',
  linkKey: 'notFound.goBackHome',
- onBackButtonPress: Navigation.goBack,
+ onBackButtonPress: () => Navigation.goBack(ROUTES.HOME),
  shouldShowLink: true,
  shouldShowBackButton: true,
  onLinkPress: () => Navigation.dismissModal(),

diff --git a/src/components/DistanceRequest.js b/src/components/DistanceRequest.js
@@ -189,7 +189,7 @@ function DistanceRequest({iou, iouType, report, transaction, mapboxAccessToken,
  useEffect(updateGradientVisibility, [scrollContainerHeight, scrollContentHeight]);
 
  const navigateBack = () => {
- Navigation.goBack(isEditing ? ROUTES.getMoneyRequestConfirmationRoute(iouType, reportID) : null);
+ Navigation.goBack(isEditing ? ROUTES.getMoneyRequestConfirmationRoute(iouType, reportID) : ROUTES.HOME);
  };
 
  const navigateToNextPage = () => {

diff --git a/src/components/HeaderWithBackButton/index.js b/src/components/HeaderWithBackButton/index.js
@@ -22,7 +22,7 @@ import useKeyboardState from '../../hooks/useKeyboardState';
 function HeaderWithBackButton({
  iconFill = undefined,
  guidesCallTaskID = '',
- onBackButtonPress = () => Navigation.goBack(),
+ onBackButtonPress = () => Navigation.goBack(ROUTES.HOME),
  onCloseButtonPress = () => Navigation.dismissModal(),
  onDownloadButtonPress = () => {},
  onThreeDotsButtonPress = () => {},

diff --git a/src/libs/Navigation/Navigation.js b/src/libs/Navigation/Navigation.js
@@ -97,7 +97,7 @@ function navigate(route = ROUTES.HOME, type) {
  * @param {Bool} shouldEnforceFallback - Enforces navigation to fallback route
  * @param {Bool} shouldPopToTop - Should we navigate to LHN on back press
  */
-function goBack(fallbackRoute = ROUTES.HOME, shouldEnforceFallback = false, shouldPopToTop = false) {
+function goBack(fallbackRoute, shouldEnforceFallback = false, shouldPopToTop = false) {
  if (!canNavigate('goBack')) {
  return;
  }

diff --git a/src/libs/Notification/PushNotification/subscribeToReportCommentPushNotifications.js b/src/libs/Notification/PushNotification/subscribeToReportCommentPushNotifications.js
@@ -27,7 +27,7 @@ export default function subscribeToReportCommentPushNotifications() {
  try {
  // If a chat is visible other than the one we are trying to navigate to, then we need to navigate back
  if (Navigation.getActiveRoute().slice(1, 2) === ROUTES.REPORT && !Navigation.isActiveRoute(`r/${reportID}`)) {
- Navigation.goBack();
+ Navigation.goBack(ROUTES.HOME);
  }
 
  Log.info('[PushNotification] onSelected() - Navigation is ready. Navigating...', false, {reportID, reportActionID});

diff --git a/src/libs/actions/App.js b/src/libs/actions/App.js
@@ -319,7 +319,7 @@ function createWorkspaceAndNavigateToIt(policyOwnerEmail = '', makeMeAdmin = fal
  .then(() => {
  if (transitionFromOldDot) {
  // We must call goBack() to remove the /transition route from history
- Navigation.goBack();
+ Navigation.goBack(ROUTES.HOME);
  }
 
  if (shouldNavigateToAdminChat) {
@@ -383,7 +383,7 @@ function setUpPoliciesAndNavigate(session, shouldNavigateToAdminChat) {
  Navigation.isNavigationReady()
  .then(() => {
  // We must call goBack() to remove the /transition route from history
- Navigation.goBack();
+ Navigation.goBack(ROUTES.HOME);
  Navigation.navigate(exitTo);
  })
  .then(endSignOnTransition);
@@ -399,7 +399,7 @@ function redirectThirdPartyDesktopSignIn() {
 
  if (url.pathname === `/${ROUTES.GOOGLE_SIGN_IN}` || url.pathname === `/${ROUTES.APPLE_SIGN_IN}`) {
  Navigation.isNavigationReady().then(() => {
- Navigation.goBack();
+ Navigation.goBack(ROUTES.HOME);
  Navigation.navigate(ROUTES.DESKTOP_SIGN_IN_REDIRECT);
  });
  }

diff --git a/src/libs/actions/IOU.js b/src/libs/actions/IOU.js
@@ -1393,14 +1393,14 @@ function deleteMoneyRequest(transactionID, reportAction, isSingleTransactionView
  // STEP 7: Navigate the user depending on which page they are on and which resources were deleted
  if (isSingleTransactionView && shouldDeleteTransactionThread && !shouldDeleteIOUReport) {
  // Pop the deleted report screen before navigating. This prevents navigating to the Concierge chat due to the missing report.
- Navigation.goBack();
+ Navigation.goBack(ROUTES.HOME);
  Navigation.navigate(ROUTES.getReportRoute(iouReport.reportID));
  return;
  }
 
  if (shouldDeleteIOUReport) {
  // Pop the deleted report screen before navigating. This prevents navigating to the Concierge chat due to the missing report.
- Navigation.goBack();
+ Navigation.goBack(ROUTES.HOME);
  Navigation.navigate(ROUTES.getReportRoute(iouReport.chatReportID));
  }
 }

diff --git a/src/libs/actions/PaymentMethods.js b/src/libs/actions/PaymentMethods.js
@@ -18,12 +18,12 @@ const kycWallRef = createRef();
  */
 function continueSetup() {
  if (!kycWallRef.current || !kycWallRef.current.continue) {
- Navigation.goBack();
+ Navigation.goBack(ROUTES.HOME);
  return;
  }
 
  // Close the screen (Add Debit Card, Add Bank Account, or Enable Payments) on success and continue with setup
- Navigation.goBack();
+ Navigation.goBack(ROUTES.HOME);
  kycWallRef.current.continue();
 }
 

diff --git a/src/libs/actions/Report.js b/src/libs/actions/Report.js
@@ -1241,7 +1241,7 @@ function updateNotificationPreferenceAndNavigate(reportID, previousValue, newVal
 function updateWelcomeMessage(reportID, previousValue, newValue) {
  // No change needed, navigate back
  if (previousValue === newValue) {
- Navigation.goBack();
+ Navigation.goBack(ROUTES.HOME);
  return;
  }
 
@@ -1261,7 +1261,7 @@ function updateWelcomeMessage(reportID, previousValue, newValue) {
  },
  ];
  API.write('UpdateWelcomeMessage', {reportID, welcomeMessage: parsedWelcomeMessage}, {optimisticData, failureData});
- Navigation.goBack();
+ Navigation.goBack(ROUTES.HOME);
 }
 
 /**
@@ -1438,7 +1438,7 @@ function deleteReport(reportID) {
  */
 function navigateToConciergeChatAndDeleteReport(reportID) {
  // Dismiss the current report screen and replace it with Concierge Chat
- Navigation.goBack();
+ Navigation.goBack(ROUTES.HOME);
  navigateToConciergeChat();
  deleteReport(reportID);
 }
@@ -1820,7 +1820,7 @@ function leaveRoom(reportID) {
  );
  Navigation.dismissModal();
  if (Navigation.getTopmostReportId() === reportID) {
- Navigation.goBack();
+ Navigation.goBack(ROUTES.HOME);
  }
  navigateToConciergeChat();
 }

diff --git a/src/pages/AddPersonalBankAccountPage.js b/src/pages/AddPersonalBankAccountPage.js
@@ -132,7 +132,7 @@ class AddPersonalBankAccountPage extends React.Component {
  this.setState({selectedPlaidAccountID});
  }}
  plaidData={this.props.plaidData}
- onExitPlaid={Navigation.goBack}
+ onExitPlaid={() => Navigation.goBack(ROUTES.HOME)}
  receivedRedirectURI={getPlaidOAuthReceivedRedirectURI()}
  selectedPlaidAccountID={this.state.selectedPlaidAccountID}
  />

diff --git a/src/pages/ConciergePage.js b/src/pages/ConciergePage.js
@@ -7,6 +7,7 @@ import ONYXKEYS from '../ONYXKEYS';
 import FullScreenLoadingIndicator from '../components/FullscreenLoadingIndicator';
 import Navigation from '../libs/Navigation/Navigation';
 import * as Report from '../libs/actions/Report';
+import ROUTES from '../ROUTES';
 
 const propTypes = {
  /** Session info for the currently logged in user. */
@@ -31,7 +32,7 @@ function ConciergePage(props) {
  useFocusEffect(() => {
  if (_.has(props.session, 'authToken')) {
  // Pop the concierge loading page before opening the concierge report.
- Navigation.goBack();
+ Navigation.goBack(ROUTES.HOME);
  Report.navigateToConciergeChat();
  } else {
  Navigation.navigate();

diff --git a/src/pages/DetailsPage.js b/src/pages/DetailsPage.js
@@ -27,8 +27,6 @@ import * as Report from '../libs/actions/Report';
 import OfflineWithFeedback from '../components/OfflineWithFeedback';
 import AutoUpdateTime from '../components/AutoUpdateTime';
 import FullPageNotFoundView from '../components/BlockingViews/FullPageNotFoundView';
-import Navigation from '../libs/Navigation/Navigation';
-import ROUTES from '../ROUTES';
 import * as UserUtils from '../libs/UserUtils';
 
 const matchType = PropTypes.shape({
@@ -130,10 +128,7 @@ function DetailsPage(props) {
  return (
  <ScreenWrapper>
  <FullPageNotFoundView shouldShow={_.isEmpty(login)}>
- <HeaderWithBackButton
- title={props.translate('common.details')}
- onBackButtonPress={() => Navigation.goBack(ROUTES.HOME)}
- />
+ <HeaderWithBackButton title={props.translate('common.details')} />
  <View
  pointerEvents="box-none"
  style={[styles.containerWithSpaceBetween]}

diff --git a/src/pages/EditRequestAmountPage.js b/src/pages/EditRequestAmountPage.js
@@ -57,10 +57,7 @@ function EditRequestAmountPage({defaultAmount, defaultCurrency, onSubmit, report
  includeSafeAreaPaddingBottom={false}
  shouldEnableMaxHeight
  >
- <HeaderWithBackButton
- title={translate('iou.amount')}
- onBackButtonPress={Navigation.goBack}
- />
+ <HeaderWithBackButton title={translate('iou.amount')} />
  <MoneyRequestAmountForm
  isEditing
  currency={defaultCurrency}

diff --git a/src/pages/EditRequestCreatedPage.js b/src/pages/EditRequestCreatedPage.js
@@ -5,7 +5,6 @@ import HeaderWithBackButton from '../components/HeaderWithBackButton';
 import Form from '../components/Form';
 import ONYXKEYS from '../ONYXKEYS';
 import styles from '../styles/styles';
-import Navigation from '../libs/Navigation/Navigation';
 import useLocalize from '../hooks/useLocalize';
 import NewDatePicker from '../components/NewDatePicker';
 
@@ -25,10 +24,7 @@ function EditRequestCreatedPage({defaultCreated, onSubmit}) {
  includeSafeAreaPaddingBottom={false}
  shouldEnableMaxHeight
  >
- <HeaderWithBackButton
- title={translate('common.date')}
- onBackButtonPress={Navigation.goBack}
- />
+ <HeaderWithBackButton title={translate('common.date')} />
  <Form
  style={[styles.flexGrow1, styles.ph5]}
  formID={ONYXKEYS.FORMS.MONEY_REQUEST_DATE_FORM}

diff --git a/src/pages/EditRequestDescriptionPage.js b/src/pages/EditRequestDescriptionPage.js
@@ -7,7 +7,6 @@ import HeaderWithBackButton from '../components/HeaderWithBackButton';
 import Form from '../components/Form';
 import ONYXKEYS from '../ONYXKEYS';
 import styles from '../styles/styles';
-import Navigation from '../libs/Navigation/Navigation';
 import CONST from '../CONST';
 import useLocalize from '../hooks/useLocalize';
 import * as Browser from '../libs/Browser';
@@ -29,10 +28,7 @@ function EditRequestDescriptionPage({defaultDescription, onSubmit}) {
  shouldEnableMaxHeight
  onEntryTransitionEnd={() => descriptionInputRef.current && descriptionInputRef.current.focus()}
  >
- <HeaderWithBackButton
- title={translate('common.description')}
- onBackButtonPress={() => Navigation.goBack()}
- />
+ <HeaderWithBackButton title={translate('common.description')} />
  <Form
  style={[styles.flexGrow1, styles.ph5]}
  formID={ONYXKEYS.FORMS.MONEY_REQUEST_DESCRIPTION_FORM}

diff --git a/src/pages/EditRequestMerchantPage.js b/src/pages/EditRequestMerchantPage.js
@@ -8,7 +8,6 @@ import HeaderWithBackButton from '../components/HeaderWithBackButton';
 import Form from '../components/Form';
 import ONYXKEYS from '../ONYXKEYS';
 import styles from '../styles/styles';
-import Navigation from '../libs/Navigation/Navigation';
 import CONST from '../CONST';
 import useLocalize from '../hooks/useLocalize';
 
@@ -40,10 +39,7 @@ function EditRequestMerchantPage({defaultMerchant, onSubmit}) {
  shouldEnableMaxHeight
  onEntryTransitionEnd={() => merchantInputRef.current && merchantInputRef.current.focus()}
  >
- <HeaderWithBackButton
- title={translate('common.merchant')}
- onBackButtonPress={Navigation.goBack}
- />
+ <HeaderWithBackButton title={translate('common.merchant')} />
  <Form
  style={[styles.flexGrow1, styles.ph5]}
  formID={ONYXKEYS.FORMS.MONEY_REQUEST_MERCHANT_FORM}

diff --git a/src/pages/EnablePayments/OnfidoStep.js b/src/pages/EnablePayments/OnfidoStep.js
@@ -13,6 +13,7 @@ import Growl from '../../libs/Growl';
 import OnfidoPrivacy from './OnfidoPrivacy';
 import walletOnfidoDataPropTypes from './walletOnfidoDataPropTypes';
 import FullPageOfflineBlockingView from '../../components/BlockingViews/FullPageOfflineBlockingView';
+import ROUTES from '../../ROUTES';
 
 const propTypes = {
  /** Stores various information used to build the UI and call any APIs */
@@ -51,7 +52,7 @@ class OnfidoStep extends React.Component {
  Growl.error(this.props.translate('onfidoStep.genericError'), 10000);
  }}
  onUserExit={() => {
- Navigation.goBack();
+ Navigation.goBack(ROUTES.HOME);
  }}
  onSuccess={(data) => {
  BankAccounts.verifyIdentity({

diff --git a/src/pages/PrivateNotes/PrivateNotesEditPage.js b/src/pages/PrivateNotes/PrivateNotesEditPage.js
@@ -24,6 +24,7 @@ import * as Report from '../../libs/actions/Report';
 import useLocalize from '../../hooks/useLocalize';
 import OfflineWithFeedback from '../../components/OfflineWithFeedback';
 import focusAndUpdateMultilineInputRange from '../../libs/focusAndUpdateMultilineInputRange';
+import ROUTES from '../../ROUTES';
 
 const propTypes = {
  /** All of the personal details for everyone */
@@ -72,7 +73,7 @@ function PrivateNotesEditPage({route, personalDetailsList, session, report}) {
  Keyboard.dismiss();
 
  // Take user back to the PrivateNotesView page
- Navigation.goBack();
+ Navigation.goBack(ROUTES.HOME);
  };
 
  return (
@@ -83,14 +84,12 @@ function PrivateNotesEditPage({route, personalDetailsList, session, report}) {
  <FullPageNotFoundView
  shouldShow={_.isEmpty(report) || _.isEmpty(report.privateNotes) || !_.has(report, ['privateNotes', route.params.accountID, 'note']) || !isCurrentUserNote}
  subtitleKey="privateNotes.notesUnavailable"
- onBackButtonPress={() => Navigation.goBack()}
  >
  <HeaderWithBackButton
  title={translate('privateNotes.title')}
  subtitle={translate('privateNotes.myNote')}
  shouldShowBackButton
  onCloseButtonPress={() => Navigation.dismissModal()}
- onBackButtonPress={() => Navigation.goBack()}
  />
  <View style={[styles.flexGrow1, styles.ph5]}>
  <View style={[styles.mb5]}>

diff --git a/src/pages/PrivateNotes/PrivateNotesListPage.js b/src/pages/PrivateNotes/PrivateNotesListPage.js
@@ -118,15 +118,11 @@ function PrivateNotesListPage({report, personalDetailsList, network, session}) {
 
  return (
  <ScreenWrapper includeSafeAreaPaddingBottom={false}>
- <FullPageNotFoundView
- shouldShow={_.isEmpty(report.reportID) || (!report.isLoadingPrivateNotes && network.isOffline && _.isEmpty(lodashGet(report, 'privateNotes', {})))}
- onBackButtonPress={() => Navigation.goBack()}
- >
+ <FullPageNotFoundView shouldShow={_.isEmpty(report.reportID) || (!report.isLoadingPrivateNotes && network.isOffline && _.isEmpty(lodashGet(report, 'privateNotes', {})))}>
  <HeaderWithBackButton
  title={translate('privateNotes.title')}
  shouldShowBackButton
  onCloseButtonPress={() => Navigation.dismissModal()}
- onBackButtonPress={() => Navigation.goBack()}
  />
  {report.isLoadingPrivateNotes && _.isEmpty(lodashGet(report, 'privateNotes', {})) ? (
  <FullScreenLoadingIndicator />

diff --git a/src/pages/PrivateNotes/PrivateNotesViewPage.js b/src/pages/PrivateNotes/PrivateNotesViewPage.js
@@ -60,14 +60,12 @@ function PrivateNotesViewPage({route, personalDetailsList, session, report}) {
  <FullPageNotFoundView
  shouldShow={_.isEmpty(report) || _.isEmpty(report.privateNotes) || !_.has(report, ['privateNotes', route.params.accountID, 'note'])}
  subtitleKey="privateNotes.notesUnavailable"
- onBackButtonPress={() => Navigation.goBack()}
  >
  <HeaderWithBackButton
  title={translate('privateNotes.title')}
  subtitle={isCurrentUserNote ? translate('privateNotes.myNote') : `${lodashGet(personalDetailsList, [route.params.accountID, 'login'], '')} note`}
  shouldShowBackButton
  onCloseButtonPress={() => Navigation.dismissModal()}
- onBackButtonPress={() => Navigation.goBack()}
  />
  <ScrollView style={[styles.flexGrow1]}>
  <OfflineWithFeedback pendingAction={lodashGet(report, ['privateNotes', route.params.accountID, 'pendingAction'], null)}>

diff --git a/src/pages/ProfilePage.js b/src/pages/ProfilePage.js
@@ -139,7 +139,7 @@ function ProfilePage(props) {
  const hasStatus = !!statusEmojiCode && Permissions.canUseCustomStatus(props.betas);
  const statusContent = `${statusEmojiCode} ${statusText}`;
 
- const navigateBackTo = lodashGet(props.route, 'params.backTo', '');
+ const navigateBackTo = lodashGet(props.route, 'params.backTo', ROUTES.HOME);
 
  const chatReportWithCurrentUser = !isCurrentUser && !Session.isAnonymousUser() ? ReportUtils.getChatByParticipants([accountID]) : 0;
 

diff --git a/src/pages/ReimbursementAccount/ReimbursementAccountPage.js b/src/pages/ReimbursementAccount/ReimbursementAccountPage.js
@@ -284,7 +284,7 @@ class ReimbursementAccountPage extends React.Component {
  const currentStep = achData.currentStep || CONST.BANK_ACCOUNT.STEP.BANK_ACCOUNT;
  const subStep = achData.subStep;
  const shouldShowOnfido = this.props.onfidoToken && !achData.isOnfidoSetupComplete;
- const backTo = lodashGet(this.props.route.params, 'backTo');
+ const backTo = lodashGet(this.props.route.params, 'backTo', ROUTES.HOME);
  switch (currentStep) {
  case CONST.BANK_ACCOUNT.STEP.BANK_ACCOUNT:
  if (this.hasInProgressVBBA()) {
@@ -405,7 +405,7 @@ class ReimbursementAccountPage extends React.Component {
  continue={this.continue}
  policyName={policyName}
  onBackButtonPress={() => {
- Navigation.goBack(lodashGet(this.props.route.params, 'backTo'));
+ Navigation.goBack(lodashGet(this.props.route.params, 'backTo', ROUTES.HOME));
  }}
  />
  );

diff --git a/src/pages/TeachersUnite/SaveTheWorldPage.js b/src/pages/TeachersUnite/SaveTheWorldPage.js
@@ -53,7 +53,7 @@ function SaveTheWorldPage(props) {
  <IllustratedHeaderPageLayout
  shouldShowBackButton
  title={translate('sidebarScreen.saveTheWorld')}
- onBackButtonPress={Navigation.goBack}
+ onBackButtonPress={() => Navigation.goBack(ROUTES.HOME)}
  backgroundColor={themeColors.PAGE_BACKGROUND_COLORS[ROUTES.I_KNOW_A_TEACHER]}
  illustration={LottieAnimations.SaveTheWorld}
  >

diff --git a/src/pages/home/ReportScreen.js b/src/pages/home/ReportScreen.js
@@ -328,7 +328,6 @@ function ReportScreen({
  subtitleKey="notFound.noAccess"
  shouldShowCloseButton={false}
  shouldShowBackButton={isSmallScreenWidth}
- onBackButtonPress={Navigation.goBack}
  shouldShowLink={false}
  >
  <OfflineWithFeedback