[BT-132] Migrate Reports - Core (Part 4)

[mjasikowski]

Description

Current State:

• 5 routes for report fields, flags, profiles, and workspace selection
• 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
EDIT_REPORT_FIELD_REQUEST	r/:reportID/edit/policyField/:policyID/:fieldID
FLAG_COMMENT	flag/:reportID/:reportActionID
PROFILE	a/:accountID
PROFILE_AVATAR	a/:accountID/avatar
NEW_REPORT_WORKSPACE_SELECTION	new-report-workspace-selection

Usages

EDIT_REPORT_FIELD_REQUEST:

• Search for usages in report field editing flow

FLAG_COMMENT:

• Search for usages in comment flagging flow

PROFILE:

• src/pages/ProfilePage.tsx (entry point)
• Multiple components navigating to user profiles

PROFILE_AVATAR:

• Search for usages in avatar viewing flow

NEW_REPORT_WORKSPACE_SELECTION:

• src/pages/NewReportWorkspaceSelectionPage.tsx:110

Files to modify:

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

Migration Progress

• EDIT_REPORT_FIELD_REQUEST
• FLAG_COMMENT
• PROFILE
• PROFILE_AVATAR
• NEW_REPORT_WORKSPACE_SELECTION

Issue Owner

Current Issue Owner: @mananjadhav

[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!