chore: related assets pills with price change#30259
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. |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 59d293d. Configure here.
🔍 Smart E2E Test Selection
click to see 🤖 AI reasoning detailsE2E Test Selection:
Performance Test Selection: |
|




Description
Adds the related assets as pills with price variation, in a sliding carousel. When the user taps a pill, it will redirect to the asset perps page.
Also fires the segment
WHATS_HAPPENING_INTERACTED/related_asset_pressedanalytics event on tap.Simulator.Screen.Recording.-.iPhone.17.Pro.-.2026-05-15.at.15.09.51.mov
Changelog
CHANGELOG entry: null
Related issues
Fixes:
Manual testing steps
Screenshots/Recordings
Before
After
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
Note
Medium Risk
Medium risk because it wires the Whats Happening carousel into the Perps live-price stream (WebSocket provider) and changes user interaction/navigation + analytics behavior; issues could surface as extra re-renders, missing data, or incorrect event firing.
Overview
Adds a horizontal related-assets pill slider to each
WhatsHappeningCard, showing per-perps-asset chips with optional 24h % change and enabling tap-to-trade behavior.Wires the homepage
WhatsHappeningSectioncarousel underPerpsStreamProviderto support live perps price subscriptions, and refactorsuseWhatsHappeningAssetPricesto acceptRelatedAsset[](updating the expanded card + tests accordingly).Introduces a new analytics interaction
related_asset_pressed(viaWHATS_HAPPENING_INTERACTED) on pill taps, and updates card/skeleton layout to move the date into the top row and replace the old aggregated asset label (e.g.,BTC +2) with individual pills.Reviewed by Cursor Bugbot for commit 4c907a9. Bugbot is set up for automated code reviews on this repo. Configure here.