fix(ramps): update entry points to use goToBuy#29631
Conversation
Co-authored-by: Cursor <cursoragent@cursor.com>
|
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. |
Order details "start new order" and card add funds now call goToBuy so unified buy V2 routes to BuildQuote or TokenSelection instead of legacy aggregator or forced deposit navigation. Co-authored-by: Cursor <cursoragent@cursor.com>
Order details sits on the legacy aggregator order flow; start new order continues to use goToAggregator. Co-authored-by: Cursor <cursoragent@cursor.com>
UB2 routes no-asset goToBuy to token selection; drop priority-token parseRampIntent helper from the card sheet. Co-authored-by: Cursor <cursoragent@cursor.com>
Map priorityToken address + caipChainId to assetId via parseRampIntent so UB2 opens BuildQuote for the funded asset when available. Co-authored-by: Cursor <cursoragent@cursor.com>
🔍 Smart E2E Test Selection
click to see 🤖 AI reasoning detailsE2E Test Selection:
The existing E2E test ( No SmokeSwap or SmokeConfirmations are needed here since the E2E test only validates the bottom sheet appearance, not the full transaction flow. The changes are unit-tested and the E2E test covers the UI surface adequately with SmokeMoney. Performance Test Selection: |
|


Description
Draft scaffold PR for TRAM-3538: update the relevant Ramps entry points to use
goToBuy()for add funds card and new order order details flows.This PR currently contains only an empty scaffold commit so implementation can start from a clean branch based on
main.Changelog
CHANGELOG entry: null
Related issues
Fixes: https://consensyssoftware.atlassian.net/browse/TRAM-3538
Manual testing steps
Screenshots/Recordings
Before
N/A - implementation pending.
After
N/A - implementation pending.
Pre-merge author checklist
Performance checks (if applicable)
Pre-merge reviewer checklist
Made with Cursor
Note
Medium Risk
Changes the card "Fund with cash" entry point from the legacy deposit navigation to unified
goToBuyrouting, which can alter which ramp flow users land in and affects analytics/tracing emitted on tap.Overview
Updates the Card add-funds bottom sheet so the cash funding option uses
useRampNavigation().goToBuy()(passing a parsed ramp intent derived from thepriorityToken) instead of the legacy deposit-specific entry point.Adjusts analytics/tracing for this entry point:
ramp_typenow reportsBUY/UNIFIED_BUY/UNIFIED_BUY_2based on unified flags, and the Sentry trace switches fromLoadDepositExperiencetoLoadRampExperiencewithrampType: BUYtags. Tests are updated to mock unified flags and assert the new navigation + event payloads.Reviewed by Cursor Bugbot for commit c2d094d. Bugbot is set up for automated code reviews on this repo. Configure here.