fix: match segment schema location enum#24842
Conversation
|
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
|
Skipping Sonar Cloud as it's looking at old code. |
6e6945d to
92c0b5c
Compare
🔍 Smart E2E Test Selection
click to see 🤖 AI reasoning detailsThis PR is a refactoring change that renames enum values in
All usages across the codebase have been updated to use the new names. The changes are purely cosmetic/naming changes with no functional impact on the swap/bridge navigation logic. The affected components include:
The risk is low because:
Selected tags cover the main affected areas:
|
|




Description
This PR:
Changelog
CHANGELOG entry: null
Related issues
Fixes: https://consensyssoftware.atlassian.net/browse/SWAPS-3774
Manual testing steps
Screenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist
Note
Updates swaps/bridge navigation and analytics to match Segment schema and simplify tracking.
SwapBridgeNavigationLocationvalues (TabBar→Main View,TokenDetails→Token View); update usages inAssetOverview,UrlAutocomplete,Wallet,WalletActions,TradeWalletActions, and related testsMain Viewtreated as navbar (omitaction_position),Token View/Rewardstracked as asset details; keep consolidatedACTION_BUTTON_CLICKED+SWAP_BUTTON_CLICKEDin hookSWAP_BUTTON_CLICKEDemission fromWalletActions(now handled in hook)useOpenSwapsdefaultlocationtoToken View; update tests accordinglyWritten by Cursor Bugbot for commit 8e1fc15. This will update automatically on new commits. Configure here.