Skip to content

feat: remove consumer rewards#880

Merged
HuberTRoy merged 3 commits intomainfrom
feat/remove-consumer-rewards
Mar 18, 2026
Merged

feat: remove consumer rewards#880
HuberTRoy merged 3 commits intomainfrom
feat/remove-consumer-rewards

Conversation

@HuberTRoy
Copy link
Copy Markdown
Member

@HuberTRoy HuberTRoy commented Mar 18, 2026

Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

Ticket: #ticket number

Test cases:

Please describe tested cases.

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Improvements (ie: code cleaning or remove unused codes or performance issue)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

UI Changes

Add before and after changes for UI if needed.

before after
0 1

Summary by CodeRabbit

  • Chores
    • Removed the alert banner from the main interface
    • Disabled the Consumer Rewards Programme card from the dashboard carousel
    • Removed the associated notification from the notification queue

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 18, 2026

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

The Consumer Rewards Programme feature is being disabled across the application. Three related changes remove the AlertBanner component from the main app layout, disable the Consumer Rewards SubqlCard from the dashboard, and remove the associated notification from the queue system.

Changes

Cohort / File(s) Summary
Consumer Rewards Programme Disabling
src/App.tsx, src/components/NotificationCentre/useMakeNotification.ts, src/pages/dashboard/components/ActiveCard/ActiveCard.tsx
Removed AlertBanner from app router, disabled Consumer Rewards SubqlCard rendering via JSX comment, and removed makeUnhealthyConsumerRewardsProjectNotification invocation from notification queue.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

A rabbit hops through code today,
Rewards cards tucked softly away,
Banners fade and queues align,
With each small change, the UI grows fine,
Springtime cleanup, clean and bright! 🐰

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The PR description is a template with no substantive content filled in, missing critical information like ticket number, test cases, type of change selection, and actual UI change images. Fill in all required template sections: specify the ticket number, describe tested cases, select and check the appropriate type of change, and add actual before/after UI screenshots.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title 'feat: remove consumer rewards' clearly and concisely summarizes the main change across all modified files.
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
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/remove-consumer-rewards
📝 Coding Plan
  • Generate coding plan for human review comments

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

@HuberTRoy HuberTRoy merged commit c26f20c into main Mar 18, 2026
1 of 2 checks passed
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