feat: Migrate useBounty detail hook to GraphQL#116
Conversation
|
@legend4tech is attempting to deploy a commit to the Threadflow Team on Vercel. A member of the Team first needs to authorize it. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review infoConfiguration used: Organization UI Review profile: CHILL Plan: Pro Disabled knowledge base sources:
📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThis PR adds descriptive documentation comments to two bounty query hooks ( Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Are U okay? |
Description
Migrates the single bounty detail fetching logic from REST to GraphQL by replacing
bountiesApi.getById(id)with the generateduseBountyQueryhook in bothuse-bounty.tsanduse-bounty-detail.ts. Both hooks now return data typed asBountyFieldsFragment, giving access to all bounty relations includingorganization,project,bountyWindow, andsubmissions.Use-bounty-detail.tshas also been renamed touse-bounty-detail.tsto follow project conventions.Changes
hooks/use-bounty.ts— replaced REST call withuseBountyQuery, added optionalenabledflag for external query controlhooks/use-bounty-detail.ts— replaced REST call withuseBountyQuery, renamed file to match conventionCloses #102
Summary by CodeRabbit