bcgov / MOH-PPPP-Enrolment

Apache License 2.0
1 stars 0 forks source link

Bug fixes #136

Closed stephanieharrison-maximus closed 2 years ago

stephanieharrison-maximus commented 2 years ago

Please check if the PR fulfills these requirements:

Please specify the type of change this PR introduces (Bug fix, feature addition, content update, chore, etc.):

Covers tickets CLMWEBDE 114-119 and 121 --adds required/optional to the end of fields as needed, code now checks if route is CSR to determine this --forces Diagnostic Code to only accept uppercase input --name validators made specifically optional to prevent errors on blur/field clear --submission page now only shows next steps if public page (not on CSR) --coveragePreAuthNumber now only runs minimum length validator if it's a public page (not CSR) --updated tests to match new requirements

Additional Notes:

stephanieharrison-maximus commented 2 years ago

Yes, Dor asked that it be there. It seemed easier to mark the first field as required on the CSR page than to go mark every single other field on the page as optional. So the "optional" for those other fields only displays on the public page and not on the CSR page