Skip to content

fix(bounty): show working state and PR guidance to claimant on claimed bounty - #729

Open
dezzy-music wants to merge 1 commit into
MergeFi:mainfrom
dezzy-music:fix/claim-button-claimant-pr-guidance-699
Open

dezzy-music wants to merge 1 commit into
MergeFi:mainfrom
dezzy-music:fix/claim-button-claimant-pr-guidance-699

Conversation

@dezzy-music

Copy link
Copy Markdown

Fixes #699

Summary

When a bounty status was claimed, ClaimButton previously showed "This bounty has already been claimed" / "Claimed by: ..." indiscriminately to all visitors, including the claimant themselves. The contributor doing the work was presented with messaging designed for lost race participants, with no actionable guidance on linking their pull request.

Changes

  • Compared bounty.claimedById with authenticated user.id (falling back to username comparison when claimedById is absent).
  • Rendered a "You're working on this bounty" state for the claimant with explicit guidance to include Fixes #<issue-number> in their pull request description to link their work and receive their automated escrow release upon merge.
  • Maintained the existing "This bounty has already been claimed" message for non-claimant visitors.
  • Added test coverage in ClaimButton.test.tsx verifying claimant detection via claimedById or claimedBy and proper rendering of PR instructions.

@vercel

vercel Bot commented Oct 13, 2026

Copy link
Copy Markdown

@dezzy-music is attempting to deploy a commit to the chonilius' projects Team on Vercel.

A member of the Team first needs to authorize it.

This branch has not been deployed

No deployments
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.

The contributor who claimed a bounty sees 'This bounty has already been claimed' with no next steps

1 participant