cal/major-domo-v2
1
0
Fork 0
Code Issues 6 Pull Requests 2 Actions Packages Projects Releases Wiki Activity

next-release #51

Merged
cal merged 2 commits from next-release into main 2026-03-01 20:56:41 +00:00
Conversation 0 Commits 2 Files Changed 3 +627 -229

2 Commits

Author SHA1 Message Date
cal
46afbf25e0 Merge pull request 'fix: roster validation includes org affiliate transactions' (#50) from fix/roster-validation-org-affiliates into next-release
All checks were successful
Build Docker Image / build (pull_request) Successful in 4m10s
Details 
Reviewed-on: #50
 2026-03-01 20:50:21 +00:00
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