Feature: admin players page is_public toggle #122

New issue

Closed

opened 2026-03-27 20:54:28 +00:00 by forgejo_admin · 0 comments

forgejo_admin commented

2026-03-27 20:54:28 +00:00

Owner

Type

Feature

Lineage

Split from westside-app#111. Frontend consumer of basketball-api#189 (PATCH visibility endpoint).

Repo

forgejo_admin/westside-app

User Story

As an admin (Marcus)
I want to toggle which players appear on the public teams page from the admin UI
So that I control roster visibility without asking a developer

Context

The admin players page exists at src/routes/(app)/admin/players/. Needs an is_public toggle column that calls PATCH /admin/players/{id}/visibility on basketball-api. Currently 78 players with is_public=False means the public teams page shows empty rosters.

File Targets

Files the agent should modify:

src/routes/(app)/admin/players/+page.svelte — add is_public toggle per player row

Files the agent should NOT touch:

Any public routes
basketball-api (separate ticket #189)

Acceptance Criteria

Each player row has a toggle/checkbox for is_public
Toggling calls PATCH /admin/players/{id}/visibility
Toggle state reflects current is_public value from API
Visual feedback on toggle success/failure

Test Expectations

Manual: toggle a player, verify /public/teams updates

Constraints

Depends on basketball-api#189 (PATCH endpoint) being merged first
No Tailwind — pure CSS vars
Follow existing admin page patterns

Checklist

PR opened
Tests pass
No unrelated changes

basketball-api#189 (API dependency)
westside-app#111 (parent ticket)
project-westside-basketball
story:WS-S6

### Type Feature ### Lineage Split from westside-app#111. Frontend consumer of basketball-api#189 (PATCH visibility endpoint). ### Repo `forgejo_admin/westside-app` ### User Story As an admin (Marcus) I want to toggle which players appear on the public teams page from the admin UI So that I control roster visibility without asking a developer ### Context The admin players page exists at `src/routes/(app)/admin/players/`. Needs an is_public toggle column that calls PATCH /admin/players/{id}/visibility on basketball-api. Currently 78 players with is_public=False means the public teams page shows empty rosters. ### File Targets Files the agent should modify: - `src/routes/(app)/admin/players/+page.svelte` — add is_public toggle per player row Files the agent should NOT touch: - Any public routes - basketball-api (separate ticket #189) ### Acceptance Criteria - [ ] Each player row has a toggle/checkbox for is_public - [ ] Toggling calls PATCH /admin/players/{id}/visibility - [ ] Toggle state reflects current is_public value from API - [ ] Visual feedback on toggle success/failure ### Test Expectations - [ ] Manual: toggle a player, verify /public/teams updates ### Constraints - Depends on basketball-api#189 (PATCH endpoint) being merged first - No Tailwind — pure CSS vars - Follow existing admin page patterns ### Checklist - [ ] PR opened - [ ] Tests pass - [ ] No unrelated changes ### Related - basketball-api#189 (API dependency) - westside-app#111 (parent ticket) - `project-westside-basketball` - `story:WS-S6`