You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Don't show a date in the campaign card is the status is upcoming
Changelog
CHANGELOG entry: null
Note
Low Risk
Low risk UI-only change that conditionally hides existing text; main risk is small visual regression if other statuses were expected to show the date separator/label.
Overview Upcoming reward campaigns no longer show a date line in the campaign tile.CampaignTile now conditionally renders the date separator and dateLabel only when campaignStatus !== 'upcoming'.
Tests were updated to assert that the campaign-tile-date-info element is not rendered for upcoming campaigns (while still present for active/complete).
Reviewed by Cursor Bugbot for commit c9a2d89. Bugbot is set up for automated code reviews on this repo. Configure here.
Selected Performance tags: None (no tests recommended)
Risk Level: low
AI Confidence: 97%
click to see 🤖 AI reasoning details
E2E Test Selection:
The PR contains two files: a minor UI fix in CampaignTile.tsx (conditionally hiding the date label for 'upcoming' campaigns) and its corresponding unit test update. The change is a simple conditional render — wrapping the bullet and date Text components in a {campaignStatus !== 'upcoming' && (...)} guard. No E2E tests reference CampaignTile, Rewards campaigns, or OndoCampaign components. The Rewards/Campaigns feature is not covered by any of the available E2E test tags. The change has no impact on shared components (navigation, modals, confirmations, controllers, Engine), no performance implications, and no cross-cutting concerns. Running any E2E tag would be unnecessary overhead with zero validation benefit for these changes.
Performance Test Selection:
The change is a simple conditional render that reduces DOM nodes for upcoming campaigns. This has negligible performance impact and does not affect any performance-sensitive paths (account lists, onboarding, login, swaps, app launch, asset loading, predictions, or perps).
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Don't show a date in the campaign card is the status is upcoming
Changelog
CHANGELOG entry: null
Note
Low Risk
Low risk UI-only change that conditionally hides existing text; main risk is small visual regression if other statuses were expected to show the date separator/label.
Overview
Upcoming reward campaigns no longer show a date line in the campaign tile.
CampaignTilenow conditionally renders the date separator anddateLabelonly whencampaignStatus !== 'upcoming'.Tests were updated to assert that the
campaign-tile-date-infoelement is not rendered for upcoming campaigns (while still present for active/complete).Reviewed by Cursor Bugbot for commit c9a2d89. Bugbot is set up for automated code reviews on this repo. Configure here.