Name and birth year should be required fields at registration #148

New Issue

2026-07-29T23:20:25Z

thatguygriff commented

2026-07-29 23:20:25 +00:00

Feedback

Mark name and birth year as required (like it does for questions)

Problem

Student name and date of birth (soon birth year — see the birth-year issue) can be submitted blank. Registration questions already have a required-field treatment; these two fields should match it.

Expected

Marked required in the UI with the same visual treatment as required registration questions
required on the inputs for client-side blocking
Server-side validation too — client-side required is not enforcement. Reject the submission with a field-level error when either is missing.

Places to touch

templates/frontend/register-page.php, templates/frontend/family-page.php
src/Auth/RegistrationPage.php — validation and error surfacing
src/Guardian/GuardianService.php — refuse blank names on edit
src/Registration/QuestionField.php — reuse its required-field markup for consistency
assets/css/frontend.css if the required marker needs styling
tests/Unit/Auth/RegistrationPageTest.php, tests/Unit/Guardian/GuardianServiceTest.php

## Feedback > Mark name and birth year as required (like it does for questions) ## Problem Student name and date of birth (soon birth year — see the birth-year issue) can be submitted blank. Registration questions already have a required-field treatment; these two fields should match it. ## Expected - Marked required in the UI with the same visual treatment as required registration questions - `required` on the inputs for client-side blocking - **Server-side validation too** — client-side `required` is not enforcement. Reject the submission with a field-level error when either is missing. ## Places to touch - `templates/frontend/register-page.php`, `templates/frontend/family-page.php` - `src/Auth/RegistrationPage.php` — validation and error surfacing - `src/Guardian/GuardianService.php` — refuse blank names on edit - `src/Registration/QuestionField.php` — reuse its required-field markup for consistency - `assets/css/frontend.css` if the required marker needs styling - `tests/Unit/Auth/RegistrationPageTest.php`, `tests/Unit/Guardian/GuardianServiceTest.php` ## Related Blocked on / pairs with the birth-year change.

thatguygriff added the bug label 2026-07-29 23:20:25 +00:00

thatguygriff referenced this issue from a commit

2026-07-30 00:00:31 +00:00

Require a name and birth year for every student

thatguygriff referenced this issue from a commit

2026-07-30 00:00:48 +00:00

Require a name and birth year for every student

thatguygriff referenced a pull request that will close this issue

2026-07-30 00:01:10 +00:00

Require a name and birth year for every student #154

thatguygriff closed this issue

2026-07-30 00:04:52 +00:00