Show a recurring lesson's policy acceptances and intake answers on every week of it #168

Merged

thatguygriff merged 1 commits from fix/167-series-policy-acceptances into main

2026-07-30 19:41:03 +00:00

Author	SHA1	Message	Date
thatguygriffandClaude Opus 5	df3462a8b3	Show a series' policy acceptances on every occurrence CI / Tests (PHP 8.2) (pull_request) Successful in 44s Details CI / Tests (PHP 8.1) (pull_request) Successful in 54s Details CI / No Debug Code (pull_request) Successful in 2s Details CI / PHPStan (pull_request) Successful in 2m54s Details CI / Coding Standards (pull_request) Successful in 2m59s Details CI / Tests (PHP 8.3) (pull_request) Successful in 2m43s Details CI / Build Plugin Zip (pull_request) Skipped Details A weekly booking reserves a series of lessons, but the student answers the intake and ticks the policy boxes once — so BookingEndpoint records both against the anchor lesson alone. The admin detail view looked them up by whichever lesson id was being viewed, so every occurrence after the first showed no answers and no acceptances at all. LessonDetail now takes the Lesson rather than a bare id and resolves the registration to `series_id ?? id`, so each occurrence reads the anchor's records. This is the same seam PaymentService already uses to find a series lesson's payment on the anchor. Nothing was ever missing from the database, so existing bookings read correctly with no migration and no schema change. Closes #167 Co-Authored-By: Claude Opus 5 <[email protected]>	2026-07-30 16:35:26 -03:00

Author

SHA1

Message

Date

thatguygriffandClaude Opus 5

df3462a8b3

Show a series' policy acceptances on every occurrence

CI / Tests (PHP 8.2) (pull_request) Successful in 44s

Details

CI / Tests (PHP 8.1) (pull_request) Successful in 54s

Details

CI / No Debug Code (pull_request) Successful in 2s

Details

CI / PHPStan (pull_request) Successful in 2m54s

Details

CI / Coding Standards (pull_request) Successful in 2m59s

Details

CI / Tests (PHP 8.3) (pull_request) Successful in 2m43s

Details

CI / Build Plugin Zip (pull_request) Skipped

Details

A weekly booking reserves a series of lessons, but the student answers
the intake and ticks the policy boxes once — so BookingEndpoint records
both against the anchor lesson alone. The admin detail view looked them
up by whichever lesson id was being viewed, so every occurrence after
the first showed no answers and no acceptances at all.

LessonDetail now takes the Lesson rather than a bare id and resolves the
registration to `series_id ?? id`, so each occurrence reads the anchor's
records. This is the same seam PaymentService already uses to find a
series lesson's payment on the anchor.

Nothing was ever missing from the database, so existing bookings read
correctly with no migration and no schema change.

Closes #167

Co-Authored-By: Claude Opus 5 <[email protected]>

2026-07-30 16:35:26 -03:00