chore(rewards): Adjust leaderboard#28932
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.
There are 2 total unresolved issues (including 1 from previous review).
❌ 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 a87cbdc. Configure here.
🔍 Smart E2E Test Selection
click to see 🤖 AI reasoning detailsE2E Test Selection:
No E2E tests exist for the Rewards feature: After thorough investigation of tests/smoke/, there is no rewards, ondo, or campaign test directory. The Rewards feature is not covered by any of the available E2E test tags. No shared components affected: The changes don't touch TabBar, navigation infrastructure, modals, confirmations, or any other shared components that could break existing E2E tests. No performance impact: No changes to list rendering, state management, startup code, or other performance-sensitive paths. Since there are no E2E tests that cover the Rewards feature and the changes are isolated to that feature area without affecting any shared infrastructure, no E2E test tags need to be run. Performance Test Selection: |
|




Description
Unused asset cleanup
Leaderboard crown icons for top 5
StatCell suffix placement
New reusable LeaderboardPositionHeader component
Audit
en.jsonAudit usage of hooks
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
Mostly UI refactors, but it changes Ondo leaderboard/stats rendering and adds additional data-fetching (portfolio/deposits) that could affect loading/error states and displayed values.
Overview
Refactors the Ondo campaign rank/tier header UI into a reusable
LeaderboardPositionHeaderand adopts it in bothOndoCampaignStatsViewandOndoLeaderboardView, changing how pending/ineligible/qualified indicators are rendered.Enhances
OndoLeaderboardrows by adding a crown icon for the top 5 entries (disabled in preview mode) and tweaks the tier selector icon.Updates data wiring/tests:
OndoLeaderboardViewnow fetches portfolio + campaign deposits to display return and computed prize pool (via new exportedgetCurrentPrize/BREAKPOINTSfromOndoPrizePool), adjustsStatCellsuffix placement, adds component/unit tests, and removes several unused rewards SVG assets and some unused i18n strings.Reviewed by Cursor Bugbot for commit 825897c. Bugbot is set up for automated code reviews on this repo. Configure here.