feat(judging): batch overview + decimal scores + delete submission#187
Merged
Conversation
…ores, delete submission - Batch overview replaces the flat chips: expand each batch to preview its submissions, see which judges are working on it (claimedBy), multi-select several unclaimed batches and 'Claim selected'. Per-submission shows which judges have saved a score (scoredBy). - Decimal scores (e.g. 4.3/5): widen score columns INTEGER -> NUMERIC(3,1) (backward-compatible), validator accepts 1dp, inputs step 0.1. - Admin-only delete submission (DELETE /:slug/submissions/:id, IDOR-guarded, scores cascade) with a trash button in the batch overview + scoring card. Server changes are additive (claimedBy/scoredBy fields); scoring write path + ballot gate unchanged. 438 server tests pass; client build + lint clean.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
…udges; fix NUMERIC string coercion - Any judge can expand any batch to see its submissions with name, VIDEO and GITHUB links, the current average score /12 (per-judge breakdown on hover), and who has scored. Batch header shows who's working on it (incl. you). - Surface per-submission 'scores' in listForJudge (additive). - Coerce NUMERIC score columns Number() in the repo transform — PostgREST returns NUMERIC as strings post-migration, which would corrupt score sums.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Bundles the judging-batch UX, decimal scores, and an admin delete. Built to not disrupt live judging — server changes are additive and the scoring write-path + ballot gate are untouched.
What's new
Batch overview (replaces the flat batch chips)
claimedByjudges) + a· MINEmarker.scoredBy), in the overview and on the scoring card.Decimal scores (e.g. 4.3/5)
INTEGER → NUMERIC(3,1)— non-destructive, existing scores preserved (5 → 5.0),CHECKranges still valid, and backward-compatible (old build keeps writing integers fine, so the migration can run first).step="0.1".Admin delete submission (for test entries)
DELETE /:slug/submissions/:submissionId(requireProgramAdmin, IDOR-guarded, scores cascade) + a trash button (admin-only) in the batch overview and scoring card. Reloads after delete so batch membership stays correct.Safety
claimedBy,scoredBy); claim/score/ballot logic unchanged.Rollout (live-safe)
20260617000000_submission_scores_decimal.sql— backward-compatible, can go anytime.