Skip to content

Commit 1c989ec

Browse files
committed
fix(workspace): re-apply Expensify#89182 back-button handler simplification
PR Expensify#87283 (Submit workspace onboarding) was merged with a stale conflict resolution that brought back the pre-Expensify#89182 multi-line onBackButtonPress handler, effectively reverting the fix for Expensify#88970 ("Back button leads to Inbox tab from workspace overview page"). Re-apply the one-liner so a backTo route is always passed to Navigation.goBack().
1 parent 016cc26 commit 1c989ec

1 file changed

Lines changed: 1 addition & 4 deletions

File tree

src/pages/workspace/WorkspaceInitialPage.tsx

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -452,10 +452,7 @@ function WorkspaceInitialPage({policyDraft, policy: policyProp, route}: Workspac
452452
>
453453
<HeaderWithBackButton
454454
title={policyName}
455-
onBackButtonPress={() =>
456-
// On narrow, goBack() pops the workspace split; goBack with a route can't resolve inside TAB_NAVIGATOR.
457-
shouldUseNarrowLayout && !route.params?.backTo ? Navigation.goBack() : Navigation.goBack(route.params?.backTo ?? ROUTES.WORKSPACES_LIST.route)
458-
}
455+
onBackButtonPress={() => Navigation.goBack(route.params?.backTo ?? ROUTES.WORKSPACES_LIST.route)}
459456
policyAvatar={policyAvatar}
460457
shouldDisplayHelpButton={shouldUseNarrowLayout}
461458
/>

0 commit comments

Comments
 (0)