fix: allow editing teams after match generation #36

Merged
dev-bot merged 11 commits from fix/edit-teams-after-match-generation into main 2026-08-20 14:49:01 -04:00
Collaborator

Summary

  • Allow existing teams to be edited after matches are generated.
  • Prevent adding or removing teams after match generation.
  • Detect the lock from generated matches rather than saved stages.
  • Add backend and frontend coverage for post-generation team edits.

Verification

  • npm run build -- --configuration development
  • npm test -- --watch=false (50 tests passed)
  • go test ./internal/backend ./migrations

Manual testing checklist

  • Sign in as an operator and create an event with at least two teams.
  • Configure a stage and generate its matches.
  • Return to the Teams page and confirm the message says teams cannot be added or removed after matches are generated.
  • Confirm the player/signup and create-team controls are unavailable, and the Remove action is unavailable.
  • Edit an existing team’s name and/or players, save it, and confirm the updated team appears in the list.
  • Reload the Teams page and confirm the team edit persisted.
  • Open the Matches page and confirm the edited team is still shown correctly in generated matches.
  • Before generating matches in a separate event, confirm teams can still be added, edited, and removed.
## Summary - Allow existing teams to be edited after matches are generated. - Prevent adding or removing teams after match generation. - Detect the lock from generated matches rather than saved stages. - Add backend and frontend coverage for post-generation team edits. ## Verification - `npm run build -- --configuration development` - `npm test -- --watch=false` (50 tests passed) - `go test ./internal/backend ./migrations` ## Manual testing checklist - [x] Sign in as an operator and create an event with at least two teams. - [x] Configure a stage and generate its matches. - [x] Return to the Teams page and confirm the message says teams cannot be added or removed after matches are generated. - [x] Confirm the player/signup and create-team controls are unavailable, and the Remove action is unavailable. - [x] Edit an existing team’s name and/or players, save it, and confirm the updated team appears in the list. - [x] Reload the Teams page and confirm the team edit persisted. - [x] Open the Matches page and confirm the edited team is still shown correctly in generated matches. - [x] Before generating matches in a separate event, confirm teams can still be added, edited, and removed.
test: update registration load key expectation
Some checks failed
PR Validation / validate (pull_request) Failing after 4m25s
7fadf943b8
Amp-Thread-ID: https://ampcode.com/threads/T-01a00004-afec-712a-a139-0e0e060908f2
Co-authored-by: Addison Emig <[email protected]>
Merge branch 'main' into fix/edit-teams-after-match-generation
Some checks failed
PR Validation / validate (pull_request) Has been cancelled
0dd4152d3d
ci: keep Orb-only sign-in out of CI
All checks were successful
PR Validation / validate (pull_request) Successful in 3m59s
8b3827ebd6
Amp-Thread-ID: https://ampcode.com/threads/T-01a00004-afec-712a-a139-0e0e060908f2
Co-authored-by: Addison Emig <[email protected]>
feat: add late players to existing teams
Some checks failed
PR Validation / validate (pull_request) Failing after 3m12s
25bc9c7ae3
Amp-Thread-ID: https://ampcode.com/threads/T-01a00004-afec-712a-a139-0e0e060908f2
Co-authored-by: Addison Emig <[email protected]>
fix: remove incorrect registration step labels
Some checks failed
PR Validation / validate (pull_request) Failing after 3m19s
3ecd17259c
Amp-Thread-ID: https://ampcode.com/threads/T-01a00004-afec-712a-a139-0e0e060908f2
Co-authored-by: Addison Emig <[email protected]>
test: avoid Orb-specific sign-in assertion
All checks were successful
PR Validation / validate (pull_request) Successful in 4m0s
e88f58e30a
Amp-Thread-ID: https://ampcode.com/threads/T-01a00004-afec-712a-a139-0e0e060908f2
Co-authored-by: Addison Emig <[email protected]>
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
kwila/tournaments!36
No description provided.