Hacktoberfest 2026:メンテナが10月に向けて印を付けた、オープンで初心者向けの issue。 Hacktoberfest の issue を見る

Locking a previously completed tile makes player and admin bingo results disagree

オープン
#16 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度
4/5
見積もり時間
3〜5日
初心者へのやさしさ
48/100
issue の種類
バグ
明瞭さ
おおむね明確
活発さ
活発
技術スタック
typescript
領域
full-stack

調査の方向性

src/routes/bingo/+page.server.ts と src/routes/admin/users/[id]/+page.server.ts のタイル処理を比較し、その後 src/lib/server/standings.ts を調査します。Issue に記載された非アクティブなタイルのシナリオを再現し、以前にマークされたロック済みタイルに対する意図されたポリシーを判断します。完了とは、プレイヤーボード、順位表、管理者の検出、検証、およびあらゆるリセット状態が、文書化された 1 つのポリシーに一貫して従うことを意味します。

索引モデルが issue の本文から書いたものです。

説明

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

  1. Let a player mark a tile that is part of a potential winning row.
  2. In /admin/tiles, change that tile to inactive.
  3. Complete the rest of that row.
  4. The player board counts the old mark and can show BINGO, but /admin/users/:id omits 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.ts
  • src/routes/admin/users/[id]/+page.server.ts
  • src/lib/server/standings.ts
主要言語
TypeScript
スター
0
フォーク
0
PR マージ指標
30日以内にマージされた PR はありません

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

hacksu/bingo のほかの issue

hacksu/bingo の issue をすべて見る

似ている issue

TypeScript の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。