fix: roster validation includes org affiliate transactions #50

Merged

cal merged 2 commits from fix/roster-validation-org-affiliates into next-release

2026-03-01 20:50:22 +00:00

Author	SHA1	Message	Date
Cal Corum	aa27769ed6	fix: roster validation now includes org affiliate transactions (closes #49 ) load_existing_transactions only queried for the base team abbreviation (e.g. "POR"), missing trades involving MiL/IL affiliates ("PORMIL", "PORIL"). This caused false "too many players" errors when a pending trade would have cleared a roster spot. - get_team_transactions now accepts Union[str, List[str]] for team_abbrev - load_existing_transactions queries all org affiliates [BASE, BASEMiL, BASEIL] - Added 5 tests covering the fix and backwards compatibility Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>	2026-03-01 10:45:21 -06:00
cal	a80addc742	Merge pull request 'next-release' (#47 ) from next-release into main All checks were successful Build Docker Image / build (push) Successful in 46s Details Reviewed-on: #47	2026-02-22 22:33:35 +00:00

Author

SHA1

Message

Date

Cal Corum

aa27769ed6

fix: roster validation now includes org affiliate transactions (closes #49 )

load_existing_transactions only queried for the base team abbreviation
(e.g. "POR"), missing trades involving MiL/IL affiliates ("PORMIL",
"PORIL"). This caused false "too many players" errors when a pending
trade would have cleared a roster spot.

- get_team_transactions now accepts Union[str, List[str]] for team_abbrev
- load_existing_transactions queries all org affiliates [BASE, BASEMiL, BASEIL]
- Added 5 tests covering the fix and backwards compatibility

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

2026-03-01 10:45:21 -06:00

cal

a80addc742

Merge pull request 'next-release' (#47 ) from next-release into main

Build Docker Image / build (push) Successful in 46s

Details

Reviewed-on: #47

2026-02-22 22:33:35 +00:00

fix: roster validation includes org affiliate transactions #50

2 Commits