Skip to content

fix: display flat for RHF peer-review bounty notifications#749

Open
jeevan6996 wants to merge 4 commits intoResearchHub:mainfrom
jeevan6996:fix/rhf-peer-review-bounty-display
Open

fix: display flat for RHF peer-review bounty notifications#749
jeevan6996 wants to merge 4 commits intoResearchHub:mainfrom
jeevan6996:fix/rhf-peer-review-bounty-display

Conversation

@jeevan6996
Copy link
Copy Markdown

@jeevan6996 jeevan6996 commented Apr 7, 2026

Summary

Adjusts bounty notification display so RHF-sponsored peer review bounty prompts show a flat $150 USD amount instead of derived variable amounts.

Closes ResearchHub/issues#342

Changes

  • add a notification-level USD override for peer-review bounty prompts (BOUNTY_FOR_YOU + REVIEW)
  • use the flat amount from FOUNDATION_BOUNTY_FLAT_USD
  • apply the same override in the notification amount badge so message and badge stay consistent

Files

  • components/Notification/lib/formatNotification.ts
  • components/Notification/NotificationItem.tsx

Testing

  • attempted to run frontend checks locally, but Node is currently broken in this environment due a missing dynamic library (libsimdjson.31.dylib), so CI validation is required.

Note

Low Risk
Small, UI-only formatting change scoped to a single notification type; risk is limited to incorrect amount/currency display for affected notifications.

Overview
Updates BOUNTY_FOR_YOU notifications with bounty_type=REVIEW to display a fixed FOUNDATION_BOUNTY_FLAT_USD amount instead of deriving USD from the RSC amount/exchange rate.

Adds getBountyForYouUsdOverride in formatNotification.ts and applies it both to the notification message text and the CurrencyBadge in NotificationItem (forcing USD + skipping conversion) so the badge and message remain consistent.

Reviewed by Cursor Bugbot for commit dfde721. Bugbot is set up for automated code reviews on this repo. Configure here.

@jeevan6996 jeevan6996 requested a review from a team as a code owner April 7, 2026 07:14
@chatgpt-codex-connector
Copy link
Copy Markdown

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

@vercel
Copy link
Copy Markdown
Contributor

vercel bot commented Apr 7, 2026

@jeevan6996 is attempting to deploy a commit to the ResearchHub Team on Vercel.

A member of the Team first needs to authorize it.

Comment thread components/Notification/NotificationItem.tsx Outdated
Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit dfde721. Configure here.

Comment thread components/Notification/lib/formatNotification.ts
@jeevan6996
Copy link
Copy Markdown
Author

Implemented the Cursor feedback and pushed an update to keep bounty messaging consistent: is now applied regardless of , so the badge/value no longer diverges from the notification message in peer-review bounty cases.

@jeevan6996
Copy link
Copy Markdown
Author

Correction: implemented the Cursor feedback and pushed an update to keep bounty messaging consistent. The bountyUsdOverride value is now applied regardless of showUSD, so the badge/value no longer diverges from the notification message in peer-review bounty cases.

@cursor
Copy link
Copy Markdown

cursor bot commented Apr 11, 2026

You have used all of your free Bugbot PR reviews.

To receive reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

@sonarqubecloud
Copy link
Copy Markdown

@jeevan6996
Copy link
Copy Markdown
Author

Quick follow-up: Sonar checks are green; the remaining blocker is Vercel authorization for this repo/team integration. If a team member can authorize the deployment integration, this PR should be unblocked for normal review.

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.

RHF bounty dollar amount display

1 participant