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.
…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.
…s-delete feat(judging): batch overview + decimal scores + delete submission
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
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.
Promotes
develop→main(prod deploy). Ships the judging upgrade (#187).Included
NUMERIC(3,1); inputs step 0.1. Includes theNumber()coercion fix (PostgREST returns NUMERIC as strings).Migration
20260617000000_submission_scores_decimal.sql— backward-compatible (widening; existing integer scores preserved). Apply alongside this deploy.Safety
Additive server fields; claim/score/ballot write-path unchanged. 438 server tests pass; client build + lint clean. Scoring hasn't started, so low-risk.