Skip to content

fix(bounty): derive shortlist selection from live application list#672

Merged
0xdevcollins merged 1 commit into
feat/t-replacefrom
feat/bounty-applications-review-fixed
Jul 5, 2026
Merged

fix(bounty): derive shortlist selection from live application list#672
0xdevcollins merged 1 commit into
feat/t-replacefrom
feat/bounty-applications-review-fixed

Conversation

@0xdevcollins

@0xdevcollins 0xdevcollins commented Jul 5, 2026

Copy link
Copy Markdown
Collaborator

Summary

Follow-up fix to the applications review tab (#670, since merged into feat/t-replace). Originally this branch superseded #670; after that merge it was rebased down to just this one fix commit.

The bug

Declining a checked application unmounted its checkbox but left its id in the checked set, so:

  • the sticky-bar count stayed inflated, and
  • Approve shortlist submitted a DECLINED application's id.

The fix

The selection is derived from the live list (selectedIds = checked ids still in SUBMITTED) — the count, the button's disabled state, and the shortlist payload only ever include actionable applications.

Review notes from #670 (non-blocking, not addressed here)

  • useInvalidateApplications re-derives the 'org-applications' key literal instead of using a bountyKeys helper — drift risk.
  • retry: false on the list query deviates from the global query defaults without a comment.
  • An empty decline reason is sent as "" instead of omitted.
  • videoIntroUrl / portfolioLinks are applicant-supplied and rendered as hrefs without scheme validation (same pattern as the submissions panel — candidate for a shared helper).

Verification

tsc --noEmit — 0 errors on this head (rebased onto current feat/t-replace).

🤖 Generated with Claude Code

@vercel

vercel Bot commented Jul 5, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
boundless-kd16 Ready Ready Preview, Comment Jul 5, 2026 1:11pm

@coderabbitai

coderabbitai Bot commented Jul 5, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 4ada30ac-fffc-4754-ba0f-aaeefad58ff9

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/bounty-applications-review-fixed

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Declining a checked application unmounted its checkbox but left its id in
the checked set, so the sticky-bar count stayed inflated and Approve
shortlist submitted a DECLINED application. Filter the selection against
the current SUBMITTED rows so the count and the payload only ever include
actionable applications.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@0xdevcollins 0xdevcollins force-pushed the feat/bounty-applications-review-fixed branch from b5b4423 to e2c1bef Compare July 5, 2026 13:08
@0xdevcollins 0xdevcollins changed the title feat(bounty): applications review tab (#631) + shortlist selection fix fix(bounty): derive shortlist selection from live application list Jul 5, 2026
@0xdevcollins 0xdevcollins merged commit 067746e into feat/t-replace Jul 5, 2026
2 of 3 checks passed
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