Admin review and standings still use a 5x5 card after card-size generalization
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức phù hợp với người mới
- 78/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức độ hoạt động
- Sôi nổi
- Công nghệ
- typescript
- Lĩnh vực
- full-stack
Hướng nghiên cứu
Bắt đầu với src/routes/bingo/+page.server.ts作为 triển khai tham chiếu chính xác, sau đó so sánh cách kích thước thẻ và kích thước pool hiệu dụng được truyền qua các tệp admin, standings, leaderboard và Svelte bị ảnh hưởng. Hoàn tất khi admin review và standings phát hiện bingo trên các thẻ lớn hơn, lưới admin sử dụng kích thước được hiển thị và giá trị tối đa của leaderboard phản ánh kích thước đó.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Problem
The player card calculates cardSize from effectivePoolSize(tiles) and passes it to shuffleTilesForUser and detectBingo. The admin review page and loadStandings do not: both call the functions with their 5x5 defaults. The admin Svelte view also hard-codes GRID_SIZE.
Reproduction
- Configure at least 49 effective tiles, so the player card becomes 7x7.
- Have a player complete a row on the displayed 7x7 card.
- Open that player in
/admin/users/:id, or view the leaderboard.
The player sees a bingo, but the admin and standings calculation only evaluates a 5x5 subset and can report no bingo, blocking verification. The leaderboard also advertises a 24-tile maximum even for larger cards.
Suggested fix
Compute the effective pool size and card size once in each consumer (or centralize board-state construction), then pass that size through shuffleTilesForUser, detectBingo, and the admin grid and leaderboard maximum.
Affected code
src/routes/admin/users/[id]/+page.server.tssrc/routes/admin/users/[id]/+page.sveltesrc/lib/server/standings.tssrc/routes/leaderboard/+page.server.tssrc/routes/bingo/+page.server.ts(the correct reference implementation)
- Ngôn ngữ chính
- TypeScript
- Star
- 0
- Fork
- 0
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Chuẩn bị môi trường
- Có Dockerfile hoặc tệp Docker Compose
- Không có mẫu pull request
- Không có hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của hacksu/bingo
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 48/100
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 72/100
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 68/100
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 68/100
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 72/100
Issue tương tự
-
area/frontend area/v2 kind/bug priority/needs-triage
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
kubeflow/notebooks#1498 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
Maintainer thường phản hồi trong vòng 1 ngày
-
P1
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 66/100
SuruchBoss/Cwork#90 ·
-
bug cli service
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 85/100
521xueweihan/HelloGitHub#3922 ·