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

Public site: strip stale tryout content + fix practice count (2 not 3) #75

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

Type

Feature

Lineage

Standalone — no plan phase. Content cleanup on live public site.

Repo

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

User Story

As a prospective parent visiting the public site,
I want to see accurate program information (no stale tryout ads, correct practice schedule)
So that I trust the program is organized and current

Context

Marcus actively shares the GitHub Pages site with recruits and families. The site currently promotes tryouts that are cancelled (March 13 date), shows "3 practices/week" (now 2), and has Stripe payment links for tryout registration that shouldn't be live. This is urgent — every day the site lies about the program.

File Targets

Files to modify:

  • index.html — remove tryout banner (lines 52-59), remove tryout section (lines 109-140), fix practice count in About card (line 90) and FAQ (line 186), update "How It Works" header (line 147), replace bottom CTA (lines 209-215), remove hero Stripe CTA (line 67)

Files NOT to touch:

  • css/style.css — no style changes needed for removal
  • js/main.js — no JS changes needed

Acceptance Criteria

  • Tryout announcement banner removed
  • Tryout details section (#tryouts) removed
  • All "Register Now" CTAs and Stripe links removed
  • About card: "Three 2-hour practices" → "Two 2-hour practices per week"
  • FAQ: practice commitment answer updated from 3 to 2
  • Bottom CTA: replaced with general "Join Westside" or season info messaging
  • Nav: "Tryouts" link removed
  • No references to tryouts remain anywhere on the page
  • OG/meta description updated to remove tryout references

Test Expectations

  • Visual: open index.html on phone — no tryout content visible
  • Search page source for "tryout" / "Tryouts" / "Register Now" — zero hits
  • All remaining nav links work
  • No automated tests (static HTML)

Constraints

  • No frameworks, no build tools — plain HTML/CSS/JS
  • Mobile-first (90%+ of users on phone)
  • Playground page approved on phone before copying to production repo
  • Keep existing design patterns intact

Checklist

  • Playground prototype built
  • Approved on phone
  • Copied to west-side-basketball
  • Pushed to GitHub Pages
  • Verified live

Related

  • project-westside-basketball — project page
  • Issue #71 (westside-app) — practice schedule page (related future work)
### Type Feature ### Lineage Standalone — no plan phase. Content cleanup on live public site. ### Repo `ldraney/west-side-basketball` (GitHub Pages) Prototype in `forgejo_admin/westside-playground` ### User Story As a prospective parent visiting the public site, I want to see accurate program information (no stale tryout ads, correct practice schedule) So that I trust the program is organized and current ### Context Marcus actively shares the GitHub Pages site with recruits and families. The site currently promotes tryouts that are cancelled (March 13 date), shows "3 practices/week" (now 2), and has Stripe payment links for tryout registration that shouldn't be live. This is urgent — every day the site lies about the program. ### File Targets Files to modify: - `index.html` — remove tryout banner (lines 52-59), remove tryout section (lines 109-140), fix practice count in About card (line 90) and FAQ (line 186), update "How It Works" header (line 147), replace bottom CTA (lines 209-215), remove hero Stripe CTA (line 67) Files NOT to touch: - `css/style.css` — no style changes needed for removal - `js/main.js` — no JS changes needed ### Acceptance Criteria - [ ] Tryout announcement banner removed - [ ] Tryout details section (#tryouts) removed - [ ] All "Register Now" CTAs and Stripe links removed - [ ] About card: "Three 2-hour practices" → "Two 2-hour practices per week" - [ ] FAQ: practice commitment answer updated from 3 to 2 - [ ] Bottom CTA: replaced with general "Join Westside" or season info messaging - [ ] Nav: "Tryouts" link removed - [ ] No references to tryouts remain anywhere on the page - [ ] OG/meta description updated to remove tryout references ### Test Expectations - [ ] Visual: open index.html on phone — no tryout content visible - [ ] Search page source for "tryout" / "Tryouts" / "Register Now" — zero hits - [ ] All remaining nav links work - No automated tests (static HTML) ### Constraints - No frameworks, no build tools — plain HTML/CSS/JS - Mobile-first (90%+ of users on phone) - Playground page approved on phone before copying to production repo - Keep existing design patterns intact ### Checklist - [ ] Playground prototype built - [ ] Approved on phone - [ ] Copied to west-side-basketball - [ ] Pushed to GitHub Pages - [ ] Verified live ### Related - `project-westside-basketball` — project page - Issue #71 (westside-app) — practice schedule page (related future work)
forgejo_admin commented 2026-03-26 03:59:11 +00:00
Author
Owner
Copy link

Scope Review: NEEDS_REFINEMENT

Review note: review-354-2026-03-25

File targets in index.html are accurate (all line numbers verified), but blast radius extends to sibling files not scoped in the ticket.

  • staff.html and sponsors.html both have nav links to #tryouts and staff.html has a bottom CTA pointing to tryouts — these become broken anchors after the #tryouts section is removed
  • success.html is an entire tryout registration confirmation page that becomes orphaned — needs explicit scope (remove/redirect) or deferral
  • Acceptance criterion "No references to tryouts remain anywhere on the page" should expand to cover all .html files, or explicitly state which files are in-scope vs deferred
## Scope Review: NEEDS_REFINEMENT Review note: `review-354-2026-03-25` File targets in index.html are accurate (all line numbers verified), but blast radius extends to sibling files not scoped in the ticket. - **staff.html** and **sponsors.html** both have nav links to `#tryouts` and staff.html has a bottom CTA pointing to tryouts — these become broken anchors after the #tryouts section is removed - **success.html** is an entire tryout registration confirmation page that becomes orphaned — needs explicit scope (remove/redirect) or deferral - Acceptance criterion "No references to tryouts remain anywhere on the page" should expand to cover all .html files, or explicitly state which files are in-scope vs deferred
forgejo_admin commented 2026-03-26 04:00:05 +00:00
Author
Owner
Copy link

Scope refinement based on review-354-2026-03-25:

Expanding file targets to cover all HTML files with tryout references:

Updated File Targets

Files to modify:

  • index.html — remove tryout banner, tryout section, hero/bottom CTAs, fix practice count, update How It Works, remove Tryouts nav link, update meta descriptions
  • staff.html — remove Tryouts nav link (line 44), replace bottom CTA "View Tryouts Info" (lines 149-150) with general messaging
  • sponsors.html — remove Tryouts nav link (line 44)
  • success.html — this is an orphaned tryout confirmation page. Either remove entirely or repurpose as a general contact confirmation page.

Updated Acceptance Criteria

  • All original criteria remain
  • Added: No references to tryouts remain in ANY .html file in the repo (not just index.html)
  • Added: Nav is consistent across all pages (no Tryouts link on any page)
  • Added: success.html either removed or repurposed

Scope boundary with #77

This ticket handles tryout REMOVAL across all pages. Ticket #77 handles ADDING the schedule page + Schedule nav link. Clean separation.

**Scope refinement based on review-354-2026-03-25:** Expanding file targets to cover all HTML files with tryout references: ### Updated File Targets Files to modify: - `index.html` — remove tryout banner, tryout section, hero/bottom CTAs, fix practice count, update How It Works, remove Tryouts nav link, update meta descriptions - `staff.html` — remove Tryouts nav link (line 44), replace bottom CTA "View Tryouts Info" (lines 149-150) with general messaging - `sponsors.html` — remove Tryouts nav link (line 44) - `success.html` — this is an orphaned tryout confirmation page. Either remove entirely or repurpose as a general contact confirmation page. ### Updated Acceptance Criteria - All original criteria remain - **Added**: No references to tryouts remain in ANY .html file in the repo (not just index.html) - **Added**: Nav is consistent across all pages (no Tryouts link on any page) - **Added**: success.html either removed or repurposed ### Scope boundary with #77 This ticket handles tryout REMOVAL across all pages. Ticket #77 handles ADDING the schedule page + Schedule nav link. Clean separation.
forgejo_admin commented 2026-03-26 04:03:45 +00:00
Author
Owner
Copy link

Repo convention note: Issue filed on westside-app as tracking repo for static site work. Actual target: ldraney/west-side-basketball (GitHub Pages) via svelte-playground/westside/ prototype. west-side-basketball has no Forgejo instance, so westside-app serves as the issue tracker for all public site work.

**Repo convention note:** Issue filed on westside-app as tracking repo for static site work. Actual target: `ldraney/west-side-basketball` (GitHub Pages) via `svelte-playground/westside/` prototype. west-side-basketball has no Forgejo instance, so westside-app serves as the issue tracker for all public site work.
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
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-app#75
No description provided.