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

fix: update Queens practice schedule Fri BWill → Wed Granger (#40) #41

Merged
forgejo_admin merged 3 commits from 40-fix-queens-practice-schedule into main 2026-04-04 22:01:36 +00:00
Conversation 1 Commits 3 Files changed 2 +19 -4
forgejo_admin commented 2026-04-04 21:46:29 +00:00
Contributor
Copy link

Summary

Update the Queens (girls travel) practice schedule in the contract page from Friday at BWill 7-9 PM to Wednesday at Granger High School 6:30-7:30 PM.

Changes

  • src/routes/contract/[token]/+page.svelte -- Changed Queens practice day from Friday to Wednesday, location from BWill to Granger High School, time from 7-9 PM to 6:30-7:30 PM

Test Plan

  • npm test -- 14/14 tests pass
  • npm run check -- pre-existing type errors only (missing @types/node, @types/pg), no new issues
  • Verified boys travel and local Kings Friday/BWill references are untouched

Review Checklist

  • Only Queens practice schedule modified
  • Boys travel and local Kings sections untouched
  • HTML entities match surrounding code style (–)
  • Tests pass

Related

Closes #40

Related Notes

None

## Summary Update the Queens (girls travel) practice schedule in the contract page from Friday at BWill 7-9 PM to Wednesday at Granger High School 6:30-7:30 PM. ## Changes - `src/routes/contract/[token]/+page.svelte` -- Changed Queens practice day from Friday to Wednesday, location from BWill to Granger High School, time from 7-9 PM to 6:30-7:30 PM ## Test Plan - `npm test` -- 14/14 tests pass - `npm run check` -- pre-existing type errors only (missing @types/node, @types/pg), no new issues - Verified boys travel and local Kings Friday/BWill references are untouched ## Review Checklist - [x] Only Queens practice schedule modified - [x] Boys travel and local Kings sections untouched - [x] HTML entities match surrounding code style (`–`) - [x] Tests pass ## Related Closes #40 ## Related Notes None
- Add queens-active CSS class that swaps --color-red to #e91e8c (pink)
- Apply queens-active on html element when isGirls is true
- Rename Nike Tournament to Nike EYBL Tournament
- Update Live Periods section: "best tournament in the country"

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Was nested inside :root causing :root :root.queens-active selector
which never matches. Now at top level as :root.queens-active.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Change girls travel (Queens) practice from Friday at BWill 7-9 PM
to Wednesday at Granger High School 6:30-7:30 PM.

Closes #40

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
forgejo_admin commented 2026-04-04 21:47:58 +00:00
Author
Contributor
Copy link

QA Review

Scope Check

  • PR targets exactly the Queens (girls travel) practice schedule section
  • Only 1 file changed, 2 insertions, 2 deletions
  • Boys travel and local Kings Friday/BWill references confirmed untouched

Changes Verified

  • Day: Friday -> Wednesday
  • Location/Time: BWill, 7 – 9 PM -> Granger High School, 6:30 – 7:30 PM
  • HTML entities (&ndash;) match surrounding code style

Tests

  • 14/14 unit tests pass
  • svelte-check shows only pre-existing type errors (missing @types/node, @types/pg) -- no regressions

Note on Forgejo Diff

The Forgejo diff shows additional changes (app.css Queens pink scheme, Nike EYBL text updates) that are NOT part of this branch. These are from commits already merged to main on the remote. The actual branch diff (git diff main..HEAD) confirms only the 2-line practice schedule change.

Nits

None.

VERDICT: APPROVED

## QA Review ### Scope Check - PR targets exactly the Queens (girls travel) practice schedule section - Only 1 file changed, 2 insertions, 2 deletions - Boys travel and local Kings Friday/BWill references confirmed untouched ### Changes Verified - **Day:** Friday -> Wednesday - **Location/Time:** BWill, 7 – 9 PM -> Granger High School, 6:30 – 7:30 PM - HTML entities (`&ndash;`) match surrounding code style ### Tests - 14/14 unit tests pass - `svelte-check` shows only pre-existing type errors (missing @types/node, @types/pg) -- no regressions ### Note on Forgejo Diff The Forgejo diff shows additional changes (app.css Queens pink scheme, Nike EYBL text updates) that are NOT part of this branch. These are from commits already merged to `main` on the remote. The actual branch diff (`git diff main..HEAD`) confirms only the 2-line practice schedule change. ### Nits None. **VERDICT: APPROVED**
forgejo_admin deleted branch 40-fix-queens-practice-schedule 2026-04-04 22:01:36 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
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-contracts!41
No description provided.