Skip to content

fix: include creator in bounty notifications - #3815

Open
jeevan6996 wants to merge 1 commit into
ResearchHub:mainfrom
jeevan6996:fix/bounty-notification-creator-id
Open

fix: include creator in bounty notifications#3815
jeevan6996 wants to merge 1 commit into
ResearchHub:mainfrom
jeevan6996:fix/bounty-notification-creator-id

Conversation

@jeevan6996

Copy link
Copy Markdown

Summary

Expose the bounty creator ID in BOUNTY_FOR_YOU notification metadata so clients can distinguish ResearchHub Foundation bounties from user-created bounties.

Supports ResearchHub/issues#342 and ResearchHub/web#749.

Changes

  • add bounty_creator_id to both BOUNTY_FOR_YOU notification creation paths
  • assert the creator ID in the existing qualified-user notification test

Testing

  • .venv/bin/ruff format --check --diff src/reputation/tasks.py src/researchhub_comment/tests/test_comments.py
  • .venv/bin/ruff check src/reputation/tasks.py src/researchhub_comment/tests/test_comments.py
  • python manage.py test researchhub_comment.tests.test_comments.CommentViewTests.test_notify_qualified_users_about_bounty --keepdb

@jeevan6996
jeevan6996 requested a review from a team as a code owner July 18, 2026 19:13
@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.

@sonarqubecloud

Copy link
Copy Markdown

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.

1 participant