forgejo_admin/westside-landing
1
0
Fork 1
Code Issues 16 Pull requests 2 Projects Releases Packages Wiki Activity Actions

Public site: add Coach Manny to staff page #76

New issue
Closed
opened 2026-03-25 13:38:45 +00:00 by forgejo_admin · 4 comments
forgejo_admin commented 2026-03-25 13:38:45 +00:00
Owner
Copy link

Type

Feature

Lineage

Standalone — no plan phase. Staff page content update.

Repo

ldraney/west-side-basketball (GitHub Pages)
Prototype in forgejo_admin/westside-playground

User Story

As a prospective parent visiting the staff page,
I want to see all current coaches including Coach Manny
So that I know who is coaching my child

Context

Coach Manny is on staff but missing from the public staff page. The site should reflect the full coaching team. Uses the same .coach-card layout pattern as existing coach entries on staff.html.

File Targets

Files to modify:

  • staff.html — add Coach Manny card using existing .coach-card pattern

Files NOT to touch:

  • css/style.css — existing .coach-card styles sufficient
  • index.html — separate issue

Acceptance Criteria

  • Coach Manny card added to staff.html
  • Same card format as existing coaches: photo, title, credentials, philosophy
  • Photo referenced from MinIO CDN
  • Card renders correctly on mobile

Test Expectations

  • Visual: open staff.html on phone — Manny card visible and formatted correctly
  • Photo loads without broken image
  • No automated tests (static HTML)

Constraints

  • Match existing .coach-card pattern exactly
  • Mobile-first layout
  • Photo needs to be in MinIO first
  • Playground approved on phone before production copy

Checklist

  • Manny photo uploaded to MinIO
  • Playground prototype built
  • Approved on phone
  • Copied to west-side-basketball
  • Pushed to GitHub Pages

Related

  • project-westside-basketball — project page
### Type Feature ### Lineage Standalone — no plan phase. Staff page content update. ### Repo `ldraney/west-side-basketball` (GitHub Pages) Prototype in `forgejo_admin/westside-playground` ### User Story As a prospective parent visiting the staff page, I want to see all current coaches including Coach Manny So that I know who is coaching my child ### Context Coach Manny is on staff but missing from the public staff page. The site should reflect the full coaching team. Uses the same `.coach-card` layout pattern as existing coach entries on staff.html. ### File Targets Files to modify: - `staff.html` — add Coach Manny card using existing `.coach-card` pattern Files NOT to touch: - `css/style.css` — existing `.coach-card` styles sufficient - `index.html` — separate issue ### Acceptance Criteria - [ ] Coach Manny card added to staff.html - [ ] Same card format as existing coaches: photo, title, credentials, philosophy - [ ] Photo referenced from MinIO CDN - [ ] Card renders correctly on mobile ### Test Expectations - [ ] Visual: open staff.html on phone — Manny card visible and formatted correctly - [ ] Photo loads without broken image - No automated tests (static HTML) ### Constraints - Match existing `.coach-card` pattern exactly - Mobile-first layout - Photo needs to be in MinIO first - Playground approved on phone before production copy ### Checklist - [ ] Manny photo uploaded to MinIO - [ ] Playground prototype built - [ ] Approved on phone - [ ] Copied to west-side-basketball - [ ] Pushed to GitHub Pages ### Related - `project-westside-basketball` — project page
forgejo_admin commented 2026-03-26 04:02:41 +00:00
Author
Owner
Copy link

Scope Review: NEEDS_REFINEMENT

Review note: review-355-2026-03-25
Repo mismatch and missing content block this ticket from agent execution.

  • Repo mismatch: Issue filed on forgejo_admin/westside-app but target is ldraney/west-side-basketball (GitHub Pages). Agent will look at wrong codebase.
  • Photo missing: No Coach Manny photo in assets/images/ or MinIO. Checklist prerequisite unmet.
  • Photo source ambiguous: Acceptance criteria says "MinIO CDN" but all existing coaches use local assets/images/ paths.
  • Bio content missing: No role title, credentials, or philosophy text provided for Coach Manny. Agent will block or fabricate.
## Scope Review: NEEDS_REFINEMENT Review note: `review-355-2026-03-25` Repo mismatch and missing content block this ticket from agent execution. - **Repo mismatch:** Issue filed on `forgejo_admin/westside-app` but target is `ldraney/west-side-basketball` (GitHub Pages). Agent will look at wrong codebase. - **Photo missing:** No Coach Manny photo in `assets/images/` or MinIO. Checklist prerequisite unmet. - **Photo source ambiguous:** Acceptance criteria says "MinIO CDN" but all existing coaches use local `assets/images/` paths. - **Bio content missing:** No role title, credentials, or philosophy text provided for Coach Manny. Agent will block or fabricate.
forgejo_admin commented 2026-03-26 04:03:44 +00:00
Author
Owner
Copy link

Review findings (review-355-2026-03-25): NEEDS_REFINEMENT

Blocked on content from Lucas/Marcus:

  • Coach Manny's role title, credentials, and coaching philosophy (2-3 paragraphs matching existing card format)
  • Coach Manny's photo (upload to assets/images/ to match existing pattern — all 5 current coaches use local paths, NOT MinIO CDN)

Fixes needed:

  • Update acceptance criteria: photo should reference assets/images/ not MinIO CDN (matches existing pattern)
  • Bio content must be provided before this can advance to next_up

Repo note: Issue filed on westside-app as tracking repo. Actual target: ldraney/west-side-basketball (GitHub Pages). Issue body documents this. Convention: westside-app tracks static site issues since west-side-basketball has no Forgejo instance.

**Review findings (review-355-2026-03-25): NEEDS_REFINEMENT** **Blocked on content from Lucas/Marcus:** - Coach Manny's role title, credentials, and coaching philosophy (2-3 paragraphs matching existing card format) - Coach Manny's photo (upload to `assets/images/` to match existing pattern — all 5 current coaches use local paths, NOT MinIO CDN) **Fixes needed:** - Update acceptance criteria: photo should reference `assets/images/` not MinIO CDN (matches existing pattern) - Bio content must be provided before this can advance to next_up **Repo note:** Issue filed on westside-app as tracking repo. Actual target: `ldraney/west-side-basketball` (GitHub Pages). Issue body documents this. Convention: westside-app tracks static site issues since west-side-basketball has no Forgejo instance.
forgejo_admin commented 2026-03-26 15:57:45 +00:00
Author
Owner
Copy link

Status: BLOCKED

Waiting on Coach Manny — he isn't responding. Need from him:

  • Role title (e.g. "17U Local Kings Coach")
  • Bio (2-3 paragraphs matching existing coach card format: playing history, coaching experience, philosophy)
  • Photo (will go in assets/images/)

Also blocked on #85 (CSS unification) — staff page will be rebuilt on unified CSS.

Unblocking: Marcus follows up with Manny for bio + photo.

**Status: BLOCKED** Waiting on Coach Manny — he isn't responding. Need from him: - Role title (e.g. "17U Local Kings Coach") - Bio (2-3 paragraphs matching existing coach card format: playing history, coaching experience, philosophy) - Photo (will go in `assets/images/`) Also blocked on #85 (CSS unification) — staff page will be rebuilt on unified CSS. Unblocking: Marcus follows up with Manny for bio + photo.
forgejo_admin commented 2026-03-27 04:52:47 +00:00
Author
Owner
Copy link

Superseded by #98 / PR #99 (SvelteKit public site — 8 pages). Closing.

Superseded by #98 / PR #99 (SvelteKit public site — 8 pages). Closing.
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
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
forgejo_admin/westside-landing#76
No description provided.