Support league-less teams in player selection #70

New issue

Closed

opened 2026-06-03 10:32:41 +02:00 by umpire274 · 0 comments

umpire274 commented

2026-06-03 10:32:41 +02:00

(Migrated from github.com)

Support league-less teams in player selection

When a player belongs to a team with league_id = NULL, the new edit/delete flow can no longer reach that player because it only lists teams returned by Team::get_by_league. This is a supported state elsewhere: team creation still offers 0. No league (src/cli/screens/team.rs:51-54), and player CSV import auto-creates teams with league_id set to None (src/cli/screens/players.rs:480-483). In those cases users who previously could edit/delete the player now see no selectable path, so this flow needs a “No league” option or another way to include unassigned teams.

Useful? React with 👍 / 👎.

Originally posted by @chatgpt-codex-connector[bot] in https://github.com/umpire274/bs_scoring/pull/69#discussion_r3347007958

**Support league-less teams in player selection** When a player belongs to a team with `league_id = NULL`, the new edit/delete flow can no longer reach that player because it only lists teams returned by `Team::get_by_league`. This is a supported state elsewhere: team creation still offers `0. No league` (`src/cli/screens/team.rs:51-54`), and player CSV import auto-creates teams with `league_id` set to `None` (`src/cli/screens/players.rs:480-483`). In those cases users who previously could edit/delete the player now see no selectable path, so this flow needs a “No league” option or another way to include unassigned teams. Useful? React with 👍 / 👎. _Originally posted by @chatgpt-codex-connector[bot] in https://github.com/umpire274/bs_scoring/pull/69#discussion_r3347007958_