feat: add new players manually (Seydou Goudiaby + 1 TBD) with multi-team assignment #125

New issue

Closed

opened 2026-03-20 05:38:31 +00:00 by forgejo_admin · 0 comments

forgejo_admin commented

2026-03-20 05:38:31 +00:00

Owner

Type

Feature

Lineage

plan-wkq → Phase 11 → manual player additions for team announcements

Repo

forgejo_admin/basketball-api

User Story

As an admin
I want to add players who weren't registered through the normal flow
So that they appear on team rosters and receive congratulations emails

Context

Two new players need manual DB insertion:

Seydou Goudiaby — 6'6", assign to 17U Elite Kings (id=1), 17U Select Kings (id=2), 17U Local Kings (id=6)
TBD player — details incoming from Lucas, assign to 16U Select and 16U Local (NOTE: 16U Select Kings may not exist yet — may need to be created)

These players came in outside the registration flow. They need parent records, player records, and multi-team assignments via the new junction table.

File Targets

Files the agent should modify or create:

alembic/versions/xxx_add_manual_players.py — data migration to insert parent + player records and junction table entries

Files the agent should NOT touch:

Model files — no schema changes needed (junction table from prior issue)
Route files — no endpoint changes

Acceptance Criteria

When I query 17U Elite/Select/Local rosters, Seydou appears on all three
When I query 16U Select/Local rosters, second player appears on both
Both players have valid parent records with email addresses
Height and other known details are populated

Test Expectations

Integration test: verify players exist and are on correct teams
Run command: pytest tests/ -v

Constraints

Depends on many-to-many migration landing first
If 16U Select Kings team doesn't exist, create it (need coach assignment from Lucas)
Parent emails TBD — Lucas providing details

Checklist

PR opened
Tests pass
No unrelated changes

Westside Basketball — project
Depends on: many-to-many junction table issue
Issue #118 — team announcement email (depends on players being added)

### Type Feature ### Lineage `plan-wkq` → Phase 11 → manual player additions for team announcements ### Repo `forgejo_admin/basketball-api` ### User Story As an admin I want to add players who weren't registered through the normal flow So that they appear on team rosters and receive congratulations emails ### Context Two new players need manual DB insertion: 1. **Seydou Goudiaby** — 6'6", assign to 17U Elite Kings (id=1), 17U Select Kings (id=2), 17U Local Kings (id=6) 2. **TBD player** — details incoming from Lucas, assign to 16U Select and 16U Local (NOTE: 16U Select Kings may not exist yet — may need to be created) These players came in outside the registration flow. They need parent records, player records, and multi-team assignments via the new junction table. ### File Targets Files the agent should modify or create: - `alembic/versions/xxx_add_manual_players.py` — data migration to insert parent + player records and junction table entries Files the agent should NOT touch: - Model files — no schema changes needed (junction table from prior issue) - Route files — no endpoint changes ### Acceptance Criteria - [ ] When I query 17U Elite/Select/Local rosters, Seydou appears on all three - [ ] When I query 16U Select/Local rosters, second player appears on both - [ ] Both players have valid parent records with email addresses - [ ] Height and other known details are populated ### Test Expectations - [ ] Integration test: verify players exist and are on correct teams - Run command: `pytest tests/ -v` ### Constraints - Depends on many-to-many migration landing first - If 16U Select Kings team doesn't exist, create it (need coach assignment from Lucas) - Parent emails TBD — Lucas providing details ### Checklist - [ ] PR opened - [ ] Tests pass - [ ] No unrelated changes ### Related - `Westside Basketball` — project - Depends on: many-to-many junction table issue - Issue #118 — team announcement email (depends on players being added)