-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Validate A/AS level grades #3458
Conversation
ab57614
to
4d33d42
Compare
Closing temporarily - needs a couple of changes |
4d33d42
to
7e57634
Compare
The Other Qualifications flow uses several string values to identify the qualification type. Using constants to refer these is useful because: - We can see the list of valid types handled by this flow in a single place (the top of the OtherQualificationWizard class) - The relationship between the string values and the places they are used is a little easier to see when scanning, and easier to grep for.
- Remove whitespace and upcase grades before validating the form - Validate by comparing against a list of standard A/AS level grades
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work as always. I really like the sanitize_grade_where_required method. We should use that across our grade validation 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👌
Context
As a candidate
I need to… provide the required information efficiently and accurately
So that… I can complete my application
We want grade data for A/AS level qualifications to be more structured. It should remain a free text input because there are usability issues with autcompletes, radio buttons, and selects.
Changes proposed in this pull request
Guidance to review
Can be tested by adding an A/AS level in the review app.
Link to Trello card
https://trello.com/c/7sDY3Doc
Things to check