-
Notifications
You must be signed in to change notification settings - Fork 4
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
Feature branch GRO 165 #313
base: master
Are you sure you want to change the base?
Conversation
mixin: 'radio-group', | ||
validate: ['required'], | ||
legend: { | ||
className: 'visuallyhidden' |
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.
is this an existing css class name? if not, just call the class hidden, this is overly specific
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.
This is what is used for Hof components. The GDS class is govuk-visually-hidden
.
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.
went through the changed files apart form tests, looks good
…user in case email incorrect (TBC)
3510dec
to
ccba466
Compare
What?
Add new pages to form for verify email and use a different form for countries outside UK.
Why?
request by product Owners on tickets GRO-166-167 and GRO-165
How?
Updated the field.js to include new field "nonUkAddress" then updated the routes to include new url for new page "/address-nonuk" Then updated the applicant_emailer.js and caseworker_emailer.js and summary page to include the nonUkAddress field
and
create a new step: check-email in app.index.js update all the files related to check-email step to render the view check-email.html
Testing?
update test suit where possible. BDD will be update by QAT team
Screenshots (optional)
Anything Else? (optional)
Check list
here is an example commit