Skip to content

feat(registration): cap faculty selection and replace other faculty with majors#90

Open
FinleyNeilson wants to merge 1 commit into
feat/registration-reworkfrom
feat/85-faculty-major-cascading-selection
Open

feat(registration): cap faculty selection and replace other faculty with majors#90
FinleyNeilson wants to merge 1 commit into
feat/registration-reworkfrom
feat/85-faculty-major-cascading-selection

Conversation

@FinleyNeilson

@FinleyNeilson FinleyNeilson commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Cap faculty checkbox selection at MAX_FACULTIES (2), enforced server-side in actions.ts
  • Remove the "Other" faculty checkbox and otherFaculty free-text field entirely (UI, actions.ts, RegistrationDraft, stripIrrelevantFields)
  • Add an independent majors/specialisation section: up to MAX_MAJORS (4) free-text inputs, each capped at MAX_LENGTHS.major (40 chars)
  • No-JS "add another major" via a new addMajor intent that round-trips through the cookie draft, incrementing majorCount and preserving in-progress field values — mirrors the existing "back" navigation pattern
  • Map majors through to ParsedRegistrationFormSubmission via toParsedSubmission

Closes #85

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.

1 participant