Locking a previously completed tile makes player and admin bingo results disagree
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 48/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Activo
- Stack tecnológico
- typescript
- Área
- full-stack
Línea de trabajo
Compara el manejo de las casillas en src/routes/bingo/+page.server.ts y src/routes/admin/users/[id]/+page.server.ts, y luego inspecciona src/lib/server/standings.ts. Reproduce el escenario de casilla inactiva descrito en el issue y determina la política prevista para las casillas bloqueadas marcadas previamente. Se considera terminado cuando el tablero del jugador, la clasificación, la detección de administradores, la verificación y cualquier estado de restablecimiento siguen de forma coherente una única política documentada.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Problem
Inactive tiles are treated inconsistently after a player has already marked one. The player board and loadStandings count completed.has(tile.id) regardless of isActive, but the admin review loadBingoState explicitly skips every inactive tile before detecting bingo.
Consequently, if an administrator locks a tile after players marked it, a player can still see a winning line while the administrator sees no bingo and cannot verify it. The leaderboard uses the player behavior rather than the review behavior.
Reproduction
- Let a player mark a tile that is part of a potential winning row.
- In
/admin/tiles, change that tile to inactive. - Complete the rest of that row.
- The player board counts the old mark and can show BINGO, but
/admin/users/:idomits it from detection and the verify action rejects the claim.
Suggested fix
Choose and document one policy for progress on locked tiles. If old progress remains valid, remove the !t.isActive skip from admin detection. If locking is meant to invalidate progress, apply the same rule in the player board and standings, and address verification/reset state consistently.
Affected code
src/routes/bingo/+page.server.tssrc/routes/admin/users/[id]/+page.server.tssrc/lib/server/standings.ts
- Lenguaje dominante
- TypeScript
- Estrellas
- 0
- Forks
- 0
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de hacksu/bingo
-
Dificultad 3/5 1-2 días Aptitud para principiantes 72/100
-
Dificultad 4/5 3-5 días Aptitud para principiantes 68/100
-
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
-
Dificultad 3/5 1-2 días Aptitud para principiantes 72/100
-
Dificultad 3/5 1-2 días Aptitud para principiantes 78/100
Todos los issues de hacksu/bingo
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
mksglu/context-mode#1200 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
jaegertracing/jaeger-ui#4506 ·
-
area:desktop area:ui bug platform:macos
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
anthropics/claude-code#96687 ·
-
good first issue
Dificultad 1/5 Menos de una hora Aptitud para principiantes 95/100
AOSSIE-Org/DebateAI#582 · 2 comentarios ·