feat: batch sell quote details and final review modal#30157
feat: batch sell quote details and final review modal#30157infiniteflower wants to merge 8 commits into
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. |
554f2ae to
13c94fd
Compare
4791df6 to
a9b5221
Compare
6182d23 to
b54c57f
Compare
a9b5221 to
14991d2
Compare
🔍 Smart E2E Test Selection
click to see 🤖 AI reasoning detailsE2E Test Selection:
The changes are scoped to the Bridge/Swap feature area. No existing E2E tests specifically cover BatchSell flows, but SmokeSwap covers bridge/swap functionality broadly. Per tag descriptions, SmokeSwap should be paired with SmokeConfirmations since swap flows involve transaction confirmations. The NavigationService/types.ts change is purely additive (new type imports and route params added to RootStackParamList) and does not affect existing navigation routes, so it does not warrant broader test coverage beyond the swap/bridge area. No performance tests are needed as these are UI modal additions without significant rendering performance implications. Performance Test Selection: |
|



Description
Adds the next Batch Sell review flow: the final review, and supporting info modals.
This introduces reusable quote-detail UI, skeleton states for quote-dependent values, modal-to-modal replacement behavior for bottom sheets, and Redux state for Batch Sell destination tokens and per-token slippage. The final review modal now supports collapsible token quote details, minimum received and network fee explanations, and placeholder quote values while quote fetching is still being wired.
Changelog
CHANGELOG entry: Added Batch Sell review controls, quote detail modals, and final review flow
Related issues
Related: SWAPS-4439
Manual testing steps
Screenshots/Recordings
Before
N/A
After
Screen.Recording.2026-05-15.at.11.24.48.AM.mov
Pre-merge author checklist
Performance checks (if applicable)
trace()for usage andaddTokenfor an exampleFor performance guidelines and tooling, see the Performance Guide.
Pre-merge reviewer checklist