Skip to content

[Due for payment 2026-05-20] [BT-016] Remove Stale linkTo Logic and Fix Deeplink Navigation for Dynamic Routes #90254

@mjasikowski

Description

@mjasikowski

Description

Current State:

  • After an incorrect merge, stale navigation logic was reintroduced into linkTo that conflicts with the dynamic routes system
  • The removed code handled dynamic route detection and RHP-over-stacked-tab PUSH logic (findMatchingDynamicSuffix, isRhpNavigationFromStackedTab) which is no longer needed
  • This causes bugs when navigating via deeplinks to dynamic route paths — for example, clicking a deeplink to settings/wallet/add-bank-account/verify-account does not correctly update the background page

Expected State:

  • Remove the unnecessary findMatchingDynamicSuffix and isRhpNavigationFromStackedTab logic from linkTo
  • Simplify the NAVIGATE-to-PUSH promotion condition in linkTo
  • Deeplink navigation to dynamic route paths should correctly update the full screen page behind the RHP

Context:

Scope

Files:

  • App/src/libs/Navigation/linkTo.ts - Remove stale dynamic route detection logic
Issue OwnerCurrent Issue Owner: @collectioneur

Metadata

Metadata

Labels

DailyKSv2InternalRequires API changes or must be handled by Expensify staff

Type

No type
No fields configured for issues without a type.

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions