chore(predict): cp-7.66.0 remove code related to super bowl banner on carousel#26363
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. |
🔍 Smart E2E Test Selection
click to see 🤖 AI reasoning detailsE2E Test Selection:
The Carousel component is used on the wallet home screen (imported in app/components/Views/Wallet/index.tsx). The change is well-scoped and only affects the Carousel component's handling of Predict Superbowl slides. SmokeWalletPlatform is selected because:
SmokePredictions is NOT selected because:
This is a low-risk change because it's removing code (reducing complexity) rather than adding new functionality. Performance Test Selection: |
|



Description
The Carousel previously had special handling for the Predict Superbowl banner: it could replace the entire carousel with a single
PredictMarketSportCardWrapperwhen a Superbowl slide with amarketIdwas present, and hid that slide from the normal carousel. This change removes that integration.What changed:
index.tsx): Removed Predict/Superbowl imports, thepredictSuperbowlSlideandpredictSuperbowlMarketIdmemos, the early return that renderedPredictMarketSportCardWrapper, the filter that excludedPREDICT_SUPERBOWL_VARIABLE_NAMEfrom visible slides, and thehandleSportCardDismiss/handlePredictSuperbowlLoadcallbacks (including "Banner Display" tracking for the Superbowl card).index.test.tsx): Removed thePredictMarketSportCardWrappermock, thePREDICT_SUPERBOWL_VARIABLE_NAMEimport, and the entire "Carousel Predict Superbowl Integration" describe block and its five tests (render with/without marketId, props, tracking).The Carousel no longer has any Superbowl-specific behavior; any such slides from Contentful would now be treated as normal carousel slides.
Changelog
CHANGELOG entry: null
Related issues
Fixes:
Manual testing steps
Screenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist
Note
Low Risk
Removes a one-off UI code path and its tests; behavior is simplified with minimal impact outside the Predict Superbowl banner scenario.
Overview
Removes the Carousel’s special-case integration for the Predict Super Bowl banner. The carousel no longer suppresses
PREDICT_SUPERBOWL_VARIABLE_NAMEslides, no longer rendersPredictMarketSportCardWrapperas a full replacement view based onmarketId, and drops the associated dismiss/load handlers and Superbowl-specific metrics.Updates
index.test.tsxby deleting the Predict sport card mock and the entire Predict Superbowl integration test suite, leaving only the generic carousel behavior tests.Written by Cursor Bugbot for commit d585da9. This will update automatically on new commits. Configure here.