Bug: Teams page filters on program/gender but API returns division #155

Closed
opened 2026-03-28 21:31:23 +00:00 by forgejo_admin · 0 comments

Type

Bug

Lineage

Discovered during browser validation after basketball-api #211 added division field. Companion to westside-landing #143.

Repo

forgejo_admin/westside-landing

What Broke

Teams page filters use t.program === 'kings' and t.gender === 'boys' but the API returns division: "boys"/"girls". Neither program nor gender exists in the response, so all teams fail the filter and the page renders empty even though 7 teams are returned.

Line 65-66 of src/routes/(public)/teams/+page.svelte:

let kingsTeams = $derived(sortTeams(teams.filter(t => t.program === 'kings' || t.gender === 'boys')));
let queensTeams = $derived(sortTeams(teams.filter(t => t.program === 'queens' || t.gender === 'girls')));

Repro Steps

  1. Navigate to /teams
  2. API returns 7 teams with division: "boys"/"girls"
  3. Filter checks program/gender — no matches
  4. Page shows "No Kings teams available yet"

Expected Behavior

Teams filtered by division field: boys = Kings, girls = Queens.

Environment

  • Cluster/namespace: prod
  • Service version: current main

Acceptance Criteria

  • Kings tab shows boys division teams
  • Queens tab shows girls division teams
  • No regression in sorting or roster display
  • project-westside-basketball
  • forgejo_admin/basketball-api #211 — added division field to API
### Type Bug ### Lineage Discovered during browser validation after basketball-api #211 added division field. Companion to westside-landing #143. ### Repo `forgejo_admin/westside-landing` ### What Broke Teams page filters use `t.program === 'kings'` and `t.gender === 'boys'` but the API returns `division: "boys"/"girls"`. Neither `program` nor `gender` exists in the response, so all teams fail the filter and the page renders empty even though 7 teams are returned. Line 65-66 of `src/routes/(public)/teams/+page.svelte`: ```javascript let kingsTeams = $derived(sortTeams(teams.filter(t => t.program === 'kings' || t.gender === 'boys'))); let queensTeams = $derived(sortTeams(teams.filter(t => t.program === 'queens' || t.gender === 'girls'))); ``` ### Repro Steps 1. Navigate to `/teams` 2. API returns 7 teams with `division: "boys"/"girls"` 3. Filter checks `program`/`gender` — no matches 4. Page shows "No Kings teams available yet" ### Expected Behavior Teams filtered by `division` field: `boys` = Kings, `girls` = Queens. ### Environment - Cluster/namespace: prod - Service version: current main ### Acceptance Criteria - [ ] Kings tab shows boys division teams - [ ] Queens tab shows girls division teams - [ ] No regression in sorting or roster display ### Related - `project-westside-basketball` - `forgejo_admin/basketball-api #211` — added division field to API
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
forgejo_admin/westside-landing#155
No description provided.