Skip to content

bug: respect admin mode for expired tournament check-ins#268

Closed
zeeshaun wants to merge 1 commit into
5stackgg:mainfrom
zeeshaun:fix/tournament-check-in-admin-mode
Closed

bug: respect admin mode for expired tournament check-ins#268
zeeshaun wants to merge 1 commit into
5stackgg:mainfrom
zeeshaun:fix/tournament-check-in-admin-mode

Conversation

@zeeshaun
Copy link
Copy Markdown
Contributor

@lukepolo lukepolo changed the title Respect admin mode for expired tournament check-ins bug: respect admin mode for expired tournament check-ins Jun 1, 2026
@lukepolo
Copy link
Copy Markdown
Contributor

lukepolo commented Jun 2, 2026

Thanks @zeeshaun — re-landed this as #280 since this branch is on your fork and I wanted to push the review fixups directly. Your commit is preserved as-is (you're the author on it), with my changes layered on top:

  • Linked the MatchSupport notification to the match + red color (it was a bare match ID before), matching our existing match-support convention.
  • Added an explicit de-dupe so we don't re-notify when an unread MatchSupport already exists, rather than relying solely on nulling cancels_at.
  • Documented the auto-mode coin-toss, and confirmed byes can't reach this path (they resolve at the bracket level before a match row exists).

Closing in favor of #280. Appreciate the fix! 🙏

@lukepolo lukepolo closed this Jun 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Auto-forfeit when both teams not ready (needs two modes, auto and admin)

2 participants