Skip to content

Revert "Atlas-Bounty: Implement Private Earnings Dashboard (/me/earnings)"#420

Closed
0xdevcollins wants to merge 1 commit into
mainfrom
revert-419-main
Closed

Revert "Atlas-Bounty: Implement Private Earnings Dashboard (/me/earnings)"#420
0xdevcollins wants to merge 1 commit into
mainfrom
revert-419-main

Conversation

@0xdevcollins

@0xdevcollins 0xdevcollins commented Feb 28, 2026

Copy link
Copy Markdown
Collaborator

Reverts #419

Summary by CodeRabbit

  • Removals
    • Removed the earnings dashboard and related navigation entry from the application.

@vercel

vercel Bot commented Feb 28, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
boundless-kd16 Building Building Preview, Comment Feb 28, 2026 8:50pm

@coderabbitai

coderabbitai Bot commented Feb 28, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

  • Linear integration is disabled

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between c97b3f5 and cf166cf.

📒 Files selected for processing (3)
  • app/me/earnings/page.tsx
  • components/app-sidebar.tsx
  • lib/api/user/earnings.ts
💤 Files with no reviewable changes (3)
  • components/app-sidebar.tsx
  • lib/api/user/earnings.ts
  • app/me/earnings/page.tsx

📝 Walkthrough

Walkthrough

The pull request removes the entire earnings feature from the application. This includes deletion of the earnings dashboard page component, the associated API utilities and type definitions, and the navigation entry linking to the earnings section.

Changes

Cohort / File(s) Summary
Earnings Feature Removal
app/me/earnings/page.tsx, lib/api/user/earnings.ts
Removed the complete earnings dashboard page including SummaryCard, BreakdownItem, ActivityItem components and the EarningsPage orchestration logic; deleted all API utilities (getUserEarnings, claimEarning) and related TypeScript interfaces (EarningsData, EarningActivity, GetEarningsResponse, ClaimEarningRequest, ClaimEarningResponse).
Navigation Updates
components/app-sidebar.tsx
Removed the "Earnings" navigation entry and its associated IconCurrencyDollar import from the sidebar menu.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related issues

  • Issue #390: Directly addresses the removal of the earnings feature that was previously added, including the same page, API module, and navigation entry.

Possibly related PRs

  • PR #419: This PR directly reverses the changes introduced in #419, which originally added the earnings page, API utilities, and sidebar navigation entry.

Poem

🐰 The earnings page hops away,
No more coins to count today,
API calls fade from sight,
Sidebar clicks turn to night,
But the warren's still all right! 💫

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the primary change: reverting a previous implementation of the private earnings dashboard feature.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch revert-419-main

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants