Skip to content

[#28] Fix blank rounds, re-jumble, and court-count bugs#29

Merged
javaisbetterthanpython merged 3 commits into
mainfrom
fix-rounds-courts-bugs
Jun 8, 2026
Merged

[#28] Fix blank rounds, re-jumble, and court-count bugs#29
javaisbetterthanpython merged 3 commits into
mainfrom
fix-rounds-courts-bugs

Conversation

@javaisbetterthanpython

Copy link
Copy Markdown
Owner

Summary

  • Fix blank round view after round 1 by keeping the current round until regeneration succeeds and syncing round index after new rounds
  • Handle new-round-fail / new-game-fail so generating does not block later actions
  • Fix duplicate fixedPairs in localStorage cache serialization
  • Apply court capacity when computing sit-outs (min(players, courts × 4))
  • Cap generated matches to court count; fix lookahead simulation in getNextBestRound
  • Add loading/disabled UI while a round is generating

Test plan

  • yarn build passes (verified on DO dev VM)
  • Manual: 7 players / 3 courts — round 2 renders, re-jumble and edit courts work
  • yarn test:ci passes
  • yarn lint passes

Closes #28

Made with Cursor

javaisbetterthanpython and others added 3 commits June 7, 2026 22:50
Regenerate no longer drops the current round until generation succeeds,
failed rounds clear the generating flag, sit-outs respect court capacity,
and round navigation stays in sync after new rounds.

Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@javaisbetterthanpython javaisbetterthanpython merged commit 4d761eb into main Jun 8, 2026
@javaisbetterthanpython javaisbetterthanpython deleted the fix-rounds-courts-bugs branch June 8, 2026 15:57
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.

Fix blank rounds, re-jumble, and court-count bugs

1 participant