feat(hackathons): reparto de premios por escasez, regla 1-premio-por-persona y fix del marcador de timeline#65
Conversation
…persona y fix del marcador de timeline - prizedProjects: cuando participan menos proyectos que slots de premio, ahora entrega los premios más chicos primero (3 proyectos → 100k/60k/40k) en vez de tomar los slots grandes y dejar parte del pozo sin asignar. Anclado desde el final con `slots.length - ranked.length`; el caso normal (6+ proyectos) y los empates no cambian. - /hackathons: letra chica bajo la grilla de premios + modal explicativo (PrizeRulesNote) con el ejemplo de 3 proyectos. - Regla nueva compartida por todas las páginas de hackatón: "Podés sumar varios proyectos, pero solo uno por persona puede ganar premio." - Timeline: el marcador de "hoy" queda anclado al centro del nodo activo y viaja hacia el siguiente a medida que transcurre el hackatón, en vez de arrancar en el borde izquierdo de su celda (quedaba a la izquierda del punto activo). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughThe changes adjust prize-slot allocation for small participant counts, add Spanish prize-rule disclosures and an interactive explanation modal, update hackathon page composition, and reposition the active timeline marker to the center of its node. ChangesHackathon experience updates
Estimated code review effort: 3 (Moderate) | ~25 minutes Sequence Diagram(s)sequenceDiagram
participant Visitor
participant PrizeRulesNote
participant formatSats
Visitor->>PrizeRulesNote: Click “Ver condiciones”
PrizeRulesNote->>formatSats: Format example prize amounts
formatSats-->>PrizeRulesNote: Return formatted amounts
PrizeRulesNote-->>Visitor: Display animated prize rules modal
Visitor->>PrizeRulesNote: Press Escape or click close
PrizeRulesNote-->>Visitor: Close modal
Possibly related PRs
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 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 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
🧹 Nitpick comments (2)
data/hackathons/hackathons.json (1)
24-24: 🗄️ Data Integrity & Integration | 🔵 Trivial | ⚡ Quick winRule added but not yet enforced in
prizedProjects().The new rule states each person can only win with one project, but
prizedProjects()inlib/hackathons.tsstill allocates prizes per-project without deduplicating by person. The PR objectives acknowledge this gap. Would you like me to help implement the deduplication logic — e.g., tracking awarded pubkeys and skipping subsequent projects from the same person?🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@data/hackathons/hackathons.json` at line 24, Update prizedProjects() in lib/hackathons.ts to enforce the one-prize-per-person rule: track each awarded participant’s pubkey while iterating projects, skip subsequent projects belonging to an already-awarded person, and preserve the existing prize allocation behavior for eligible projects.app/hackathons/PrizeRulesNote.tsx (1)
47-128: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick winAdd ARIA dialog semantics and focus management for accessibility.
The modal lacks
role="dialog",aria-modal="true", andaria-labelledbyon the dialog container, and doesn't move focus into the modal on open or trap Tab within it. Screen readers may not announce the modal properly, and keyboard users can Tab to elements behind the overlay.♿ Suggested accessibility improvements
export default function PrizeRulesNote() { const [open, setOpen] = useState(false); + const modalRef = useRef<HTMLDivElement>(null); + const titleId = "prize-rules-title"; useScrollLock(open); useEffect(() => { if (!open) return; + // Move focus into the modal on open + modalRef.current?.focus(); const onKey = (e: KeyboardEvent) => { if (e.key === "Escape") setOpen(false); }; window.addEventListener("keydown", onKey); return () => window.removeEventListener("keydown", onKey); }, [open]);Then on the modal container:
<motion.div initial={{ opacity: 0, scale: 0.95, y: 20 }} animate={{ opacity: 1, scale: 1, y: 0 }} exit={{ opacity: 0, scale: 0.95, y: 20 }} transition={{ type: "spring", damping: 25, stiffness: 300 }} className="relative w-full max-w-md glass-strong rounded-2xl border border-border-strong overflow-hidden" + role="dialog" + aria-modal="true" + aria-labelledby={titleId} + tabIndex={-1} + ref={modalRef} >And on the heading:
- <h2 className="font-display font-bold text-lg"> + <h2 id={titleId} className="font-display font-bold text-lg">And on the trigger button:
<button type="button" onClick={() => setOpen(true)} + aria-haspopup="dialog" + aria-expanded={open} className="underline underline-offset-2 hover:text-foreground-muted transition-colors" >🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@app/hackathons/PrizeRulesNote.tsx` around lines 47 - 128, Improve the modal in the open state rendered by the AnimatePresence block: give the dialog container role="dialog", aria-modal="true", and aria-labelledby pointing to a unique id on the “Cómo se reparten los premios” heading, and add an accessible name to the trigger button. Move focus into the dialog when it opens, trap Tab and Shift+Tab within its interactive elements, and restore focus to the trigger when it closes, using refs and effect/keyboard handling in the component.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Nitpick comments:
In `@app/hackathons/PrizeRulesNote.tsx`:
- Around line 47-128: Improve the modal in the open state rendered by the
AnimatePresence block: give the dialog container role="dialog",
aria-modal="true", and aria-labelledby pointing to a unique id on the “Cómo se
reparten los premios” heading, and add an accessible name to the trigger button.
Move focus into the dialog when it opens, trap Tab and Shift+Tab within its
interactive elements, and restore focus to the trigger when it closes, using
refs and effect/keyboard handling in the component.
In `@data/hackathons/hackathons.json`:
- Line 24: Update prizedProjects() in lib/hackathons.ts to enforce the
one-prize-per-person rule: track each awarded participant’s pubkey while
iterating projects, skip subsequent projects belonging to an already-awarded
person, and preserve the existing prize allocation behavior for eligible
projects.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 50b238b5-68f5-419a-9310-2784b77857b2
📒 Files selected for processing (4)
app/hackathons/PrizeRulesNote.tsxapp/hackathons/page.tsxdata/hackathons/hackathons.jsonlib/hackathons.ts
Agrega role="dialog", aria-modal, aria-labelledby y tabIndex al panel del modal; id en el heading; aria-haspopup/aria-expanded en el trigger. Mueve el foco al diálogo al abrir y lo restaura al trigger al cerrar. Aborda el nitpick de accesibilidad de CodeRabbit. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
Gracias @coderabbitai. Los dos nitpicks vinieron dentro de un comentario agregado (sin hilos inline resolubles), así que respondo acá: 1. Accesibilidad del modal ( 2. Enforcement de "1 premio por persona" en 🤖 Addressed by Claude Code |
|
Si quieren, puedo abrir un issue de seguimiento para rastrear la implementación futura del enforcement de "1 premio por persona" en (◕‿◕) gracias por la claridad en la explicación ✅ Action performedComments resolved. Approval is disabled; enable |
…uita CTA duplicado del hero - PrizeRulesNote ahora cubre las dos condiciones: "un premio por persona" (si participás con varios proyectos solo gana el mejor, el resto queda fuera) y la regla de escasez (premios más chicos primero si hay menos de 6). El modal pasa a titularse "Condiciones de los premios" con una sección por regla más el ejemplo de 3 proyectos. - Se reutiliza PrizeRulesNote dentro de la card "Premios" de la pestaña Resultados de cada hackatón (mismo componente que /hackathons, sin duplicar). - Se elimina el botón "Inscribite gratis" del hero de /hackathons/[id] (quedaba duplicado del que ya vive en la lista de proyectos) y sus imports sin uso. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
1 participant
Qué cambia
Tres cambios sobre los hackatones, todos en
/hackathonsy/hackathons/[id].1. Reparto de premios cuando participan menos proyectos que premios
prizedProjects()enlib/hackathons.tsahora entrega los premios más chicos primero cuando hay menos proyectos rankeados que slots de premio. Con 3 proyectos participando reparte 100k/60k/40k (los últimos 3 slots) en vez de tomar 400k/250k/150k y dejar 200k sats sin asignar.slotIdx = Math.max(0, slots.length - ranked.length).2. Regla "1 premio por persona"
Regla nueva en
data/hackathons/hackathons.json(program.rules, fuente única compartida): "Podés sumar varios proyectos, pero solo uno por persona puede ganar premio." Aparece automáticamente en la tarjeta "Participación · Reglas básicas" de todas las páginas de hackatón. Verificado enai-agentsyfoundations.3. UI: letra chica + modal explicativo de premios
Nuevo componente cliente
app/hackathons/PrizeRulesNote.tsx: letra chica bajo la grilla "Estructura de premios" con un enlace "Ver cómo funciona" que abre un modal (mismo patrón visual queLoginModal: framer-motion,useScrollLock, cierre con Escape/backdrop) explicando la regla con el ejemplo de 3 proyectos.4. Fix: marcador de "hoy" en el timeline
El pip de "hoy" quedaba a la izquierda del nodo activo (parecía que el hackatón EN CURSO aún no había empezado). Los puntos de los nodos están centrados en
(i + 0.5)/n, pero el marcador usaba((activeIdx + f)/n)— el borde izquierdo de la celda. Se agregó+ 0.5para anclarlo al centro del nodo activo; ahora viaja desde el punto del hackatón activo hacia el siguiente a medida que transcurre (f: 0→1). La rama "entre hackatones" no cambia.Verificación
ai-agents,foundations).🤖 Generated with Claude Code
Summary by CodeRabbit
New Features
Bug Fixes