Skip to content

feat: add bounty discussion thread with nested replies#1237

Open
alchip wants to merge 1 commit into
SolFoundry:mainfrom
alchip:feat/discussion-thread-838-v1
Open

feat: add bounty discussion thread with nested replies#1237
alchip wants to merge 1 commit into
SolFoundry:mainfrom
alchip:feat/discussion-thread-838-v1

Conversation

@alchip
Copy link
Copy Markdown

@alchip alchip commented May 12, 2026

Summary

  • add a new BountyDiscussion section on bounty detail pages
  • implement nested reply threads using parent/child comment structure
  • add real-time refresh via polling every 5s
  • include basic moderation/spam filtering for risky link/contact patterns
  • add comments API module + React Query hooks for listing/posting comments

Why

Issue #838 requests comments/discussion threads for bounty collaboration.

Acceptance mapping

  • Comments section with nested reply threads
  • Real-time comment updates (5s polling)
  • Moderation/spam filtering (client-side guard)

Closes #838

Wallet

Wallet: 74jSPYaxEJcGEuW4jr6bqQdg2iuLChyLEuEmC8QhcQVW

@nagiexplorer88
Copy link
Copy Markdown

This mounts the discussion UI, but the backend half of #838 is still missing.

The new client always calls /api/bounties/${bountyId}/comments for list/create, but this PR only adds frontend files and does not add a backend route or persistence for that endpoint. A code search of the current repo also does not show an existing backend comments route for bounties. As submitted, the detail page can render the discussion panel, but listing comments will hit an unavailable endpoint and posting a comment cannot persist anything.

There is also no real-time transport here; refetchInterval: 5000 is polling, not real-time comment updates, and it still depends on the missing comments API. To satisfy #838, this needs the backend comments endpoints/storage and either a real-time update channel or an explicit adjustment of that acceptance path.

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.

🏭 Bounty T2: Add Bounty Comments/Discussion Thread

2 participants