[BT-105] Migrate Workspace - Tags (Part 2)

[mjasikowski]

Description

Current State:

• 5 routes for workspace tag editing and configuration
• 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
WORKSPACE_EDIT_TAGS	workspaces/:policyID/tags/:orderWeight/edit
WORKSPACE_TAG_SETTINGS	workspaces/:policyID/tag/:orderWeight/:tagName
WORKSPACE_TAG_EDIT	workspaces/:policyID/tag/:orderWeight/:tagName/edit
WORKSPACE_TAG_GL_CODE	workspaces/:policyID/tag/:orderWeight/:tagName/gl-code
WORKSPACE_TAG_APPROVER	workspaces/:policyID/tag/:orderWeight/:tagName/approver

Usages

WORKSPACE_EDIT_TAGS:

• Search for usages in tag list editing flow

WORKSPACE_TAG_SETTINGS:

• src/pages/workspace/tags/TagGLCodePage.tsx:46
• src/pages/workspace/tags/WorkspaceViewTagsPage.tsx:212

WORKSPACE_TAG_EDIT:

• Search for usages in tag editing flow

WORKSPACE_TAG_GL_CODE:

• Search for usages in tag GL code flow

WORKSPACE_TAG_APPROVER:

• Search for usages in tag approver flow

Files to modify:

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

Migration Progress

• WORKSPACE_EDIT_TAGS
• WORKSPACE_TAG_SETTINGS
• WORKSPACE_TAG_EDIT
• WORKSPACE_TAG_GL_CODE
• WORKSPACE_TAG_APPROVER

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!