ldraney/westside-app
1
0
Fork 0
Code Issues 17 Pull requests Projects Releases Packages Wiki Activity Actions

Validate: admin users page and bulk visibility on prod #195

New issue
Open
opened 2026-03-29 22:31:51 +00:00 by forgejo_admin · 0 comments
forgejo_admin commented 2026-03-29 22:31:51 +00:00
Contributor
Copy link

Type

Feature

Lineage

Post-merge validation gap — PRs #181 (admin users) and #188 (bulk visibility) merged without auth-gated prod validation.

Repo

forgejo_admin/westside-landing

User Story

As a superadmin, I want to confirm the admin users page and bulk visibility toggle work in production so that Marcus can manage users and player visibility.

Context

PRs #181 and #188 were merged and deployed (CI green). Public-facing changes validated via Playwright. But admin-gated pages require real admin credentials to validate, which automated agents can't access.

File Targets

No code changes — validation only.

Acceptance Criteria

  • /admin/users page loads with user list
  • UUID validation prevents malformed IDs
  • Self-demotion guard blocks admin from removing own role
  • /admin/players shows bulk visibility section
  • "Make All Public" / "Hide All" toggles work per team
  • Public/Hidden filter tabs show correct counts
  • Changes reflect on /public/teams after toggling

Test Expectations

  • Chrome DevTools validation with admin credentials
  • Screenshots of both pages
  • Console checked for errors

Constraints

  • Requires admin Keycloak credentials
  • Do NOT modify any data during validation — just verify UI works

Checklist

  • Admin users page validated
  • Bulk visibility validated
  • Screenshots captured

Related

  • westside-landing#127 — admin users PR
  • westside-landing#111 — bulk visibility PR
  • project-westside-basketball
### Type Feature ### Lineage Post-merge validation gap — PRs #181 (admin users) and #188 (bulk visibility) merged without auth-gated prod validation. ### Repo `forgejo_admin/westside-landing` ### User Story As a superadmin, I want to confirm the admin users page and bulk visibility toggle work in production so that Marcus can manage users and player visibility. ### Context PRs #181 and #188 were merged and deployed (CI green). Public-facing changes validated via Playwright. But admin-gated pages require real admin credentials to validate, which automated agents can't access. ### File Targets No code changes — validation only. ### Acceptance Criteria - [ ] /admin/users page loads with user list - [ ] UUID validation prevents malformed IDs - [ ] Self-demotion guard blocks admin from removing own role - [ ] /admin/players shows bulk visibility section - [ ] "Make All Public" / "Hide All" toggles work per team - [ ] Public/Hidden filter tabs show correct counts - [ ] Changes reflect on /public/teams after toggling ### Test Expectations - [ ] Chrome DevTools validation with admin credentials - [ ] Screenshots of both pages - [ ] Console checked for errors ### Constraints - Requires admin Keycloak credentials - Do NOT modify any data during validation — just verify UI works ### Checklist - [ ] Admin users page validated - [ ] Bulk visibility validated - [ ] Screenshots captured ### Related - `westside-landing#127` — admin users PR - `westside-landing#111` — bulk visibility PR - `project-westside-basketball`
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
251-docs-populate-ws-s26-prospect-user-story
252-docs-create-arch-landing-site-architectu
236-fix-jersey-image-urls
hotfix-jersey-image-urls
232-improve-public-roster-page-accuracy-and
180-fix-jersey-deadline
142-validate-coach-login-and-read-only-roste
139-fix-admin-dashboard-stats-showing-all-ze
129-validate-girls-draft-board-coach-login-a
204-auto-tag-update
deploy-update-image-tag
110-fix-harbor-push-target
No results found.
Labels
Clear labels   
domain:backend

   
domain:devops

   
domain:frontend
No labels
domain:backend
domain:devops
domain:frontend
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
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
ldraney/westside-app#195
No description provided.