Skip to content

[BT-119] Migrate Merge Transaction #83847

@mjasikowski

Description

@mjasikowski

Description

Current State:

  • 4 routes for transaction merge flow
  • 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 (4):

Route Constant Path
MERGE_TRANSACTION_LIST_PAGE merge/:transactionID
MERGE_TRANSACTION_RECEIPT_PAGE merge/:transactionID/receipt
MERGE_TRANSACTION_DETAILS_PAGE merge/:transactionID/details
MERGE_TRANSACTION_CONFIRMATION_PAGE merge/:transactionID/confirmation

Usages

MERGE_TRANSACTION_LIST_PAGE:

  • src/libs/actions/MergeTransaction.ts
  • src/pages/TransactionMerge/ReceiptReviewPage.tsx
  • src/pages/TransactionMerge/DetailsReviewPage.tsx

MERGE_TRANSACTION_RECEIPT_PAGE:

  • Search for usages in merge receipt flow

MERGE_TRANSACTION_DETAILS_PAGE:

  • Search for usages in merge details flow

MERGE_TRANSACTION_CONFIRMATION_PAGE:

  • Search for usages in merge confirmation flow

Files to modify:

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

Migration Progress

  • MERGE_TRANSACTION_LIST_PAGE
  • MERGE_TRANSACTION_RECEIPT_PAGE
  • MERGE_TRANSACTION_DETAILS_PAGE
  • MERGE_TRANSACTION_CONFIRMATION_PAGE
Issue OwnerCurrent Issue Owner: @huult

Metadata

Metadata

Labels

InternalRequires API changes or must be handled by Expensify staffMonthlyKSv2

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