[BT-145] Migrate Travel (Part 2)

[mjasikowski]

Description

Current State:

• 5 routes for travel workspace and trip details
• These routes use backTo parameter for return navigation

Implementation:

• Replace getRoute(..., backTo) with createDynamicRoute()
• Add source screens to entryScreens array
• Remove backTo parameter from route definitions
• Test forward/back navigation

Scope

Routes (5):

Route Constant	Path
TRAVEL_WORKSPACE_CONFIRMATION	travel/upgrade/workspace/confirmation
TRAVEL_WORKSPACE_ADDRESS	travel/:domain/workspace-address
TRAVEL_VERIFY_ACCOUNT	travel/verify-account
TRIP_DETAILS	r/:reportID/trip/:transactionID
TRIP_SEGMENT_DETAILS	r/:reportID/trip/:transactionID/:pnr/:sequenceIndex

Usages

TRAVEL_WORKSPACE_CONFIRMATION:

• Search for usages in travel workspace confirmation flow

TRAVEL_WORKSPACE_ADDRESS:

• Search for usages in travel workspace address flow

TRAVEL_VERIFY_ACCOUNT:

• Search for usages in travel verify account flow

TRIP_DETAILS / TRAVEL_TRIP_DETAILS:

• src/components/ReportActionItem/MoneyRequestView.tsx:1176
• src/components/ReportActionItem/TripDetailsView.tsx:152

TRIP_SEGMENT_DETAILS:

• Search for usages in trip segment details flow

Files to modify:

• src/ROUTES.ts - Update route definitions
• src/libs/Navigation/types.ts - Remove backTo from params
• Components listed above

Migration Progress

• TRAVEL_WORKSPACE_CONFIRMATION
• TRAVEL_WORKSPACE_ADDRESS
• TRAVEL_VERIFY_ACCOUNT
• TRIP_DETAILS
• TRIP_SEGMENT_DETAILS

Issue Owner

Current Issue Owner: @suneox

[melvin-bot]

This issue has not been updated in over 15 days. @mananjadhav, @mjasikowski, @huult eroding to Monthly issue.

P.S. Is everyone reading this sure this is really a near-term priority? Be brave: if you disagree, go ahead and close it out. If someone disagrees, they'll reopen it, and if they don't: one less thing to do!