Skip to content

Add query param support to pages to distinguish youth matches from senior matches#27

Merged
session567 merged 5 commits intomainfrom
bug/links-youth-matches
Mar 1, 2026
Merged

Add query param support to pages to distinguish youth matches from senior matches#27
session567 merged 5 commits intomainfrom
bug/links-youth-matches

Conversation

@session567
Copy link
Owner

@session567 session567 commented Mar 1, 2026

Closes #13
Closes #15

Description

This PR adds query param support to the pages utility to distinguish youth matches from senior matches, and restricts the links module to only run on your own team's pages.

Checklist

  • I have added tests that prove my fix is effective or that my feature works (or tests already exist)
  • I have tested my changes in Firefox
  • I have tested my changes in a Chromium-based browser
  • I have made corresponding changes to the documentation

…nior matches

- Add getCurrentSearchParams() utility to read URL search params
- Extend Page class to support queryParams option for matching URLs with specific query params
- Refactor pages object to use nested structure (e.g., pages.matches.senior.own, pages.matches.youth.own)
- Add teamPages() helper for creating own/other team page pairs
- Improve getCurrentPage() to disambiguate pages using specificity scoring
- Add mockIsPage() test helper for cleaner module tests
- Update all modules to use new page structure
@session567 session567 self-assigned this Mar 1, 2026
@session567 session567 merged commit 9a8c70c into main Mar 1, 2026
8 checks passed
@session567 session567 deleted the bug/links-youth-matches branch March 1, 2026 00:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Disable links module on other team's match list Disable links module on the youth team's matches page

1 participant