form validations and messaging for dropdowns working as per specs

NYCPlanning · Oct 29, 2024 · 289fc5e · 289fc5e
1 parent cfb56c5
commit 289fc5e
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/client/app/components/packages/projects/projects-new-information.hbs b/client/app/components/packages/projects/projects-new-information.hbs
@@ -28,8 +28,12 @@
       @placeholder='select a borough' as |borough|>
        {{borough.label}}
     </PowerSelect>
+    {{#if form.errors.borough}}
+      {{#each form.errors.borough.validation as |message|}}
+        <div class="form-error is-visible" data-test-validation-message="select a borough">{{message}}</div>
+      {{/each}}
+    {{/if}}
   </Ui::Question>
-
   <Ui::Question @required={{true}} as |Q|>
     <Q.Label data-test-projects-new-applicant-type>
       Applicant Type
@@ -48,6 +52,12 @@
       @placeholder='select applicant type' as |dcpApplicantType| >
       {{dcpApplicantType.label}}
     </PowerSelect>
+    {{#if form.errors.applicantType}}
+      {{#each form.errors.applicantType.validation as |message|}}
+          <div class="form-error is-visible"    data-test-validation-message="select an Applicant Type"> {{message}}
+        </div>
+      {{/each}}
+    {{/if}}
   </Ui::Question>
 </form.Section>
 {{/let}}

diff --git a/client/app/validations/submittable-projects-new-form.js b/client/app/validations/submittable-projects-new-form.js
@@ -120,7 +120,7 @@ export default {
       message: 'Please select a Borough',
     }),
   ],
-  dcpApplicantType: [
+  applicantType: [
     validatePresence({
       presence: true,
       message: 'Please select an Applicant Type',