Merge remote-tracking branch 'origin/develop' into feature/CSR-563

src/common-components/button-question/button-question.vue

@@ -40,7 +40,7 @@
           </div>
       </fieldset>
     </div>
-    <div class="row form-test-error">
+    <div class="row form-test-error mt-1">
         <error-message :name="groupName" v-if="!suppressError"></error-message>
     </div>
   </div>
@@ -134,7 +134,7 @@ export default {
       return answer.Name ? answer.Name : answer;
     },
     handleCheckedChanged(val) {
-      
+
       this.pushEventToGA(this.$route.query[queryStrings.FMG_PAGE], this.GaActions.CLICKED, val.value, true);
 
       if(this.isMultiSelect && this.selectedValues) {

src/common-components/textbox-question/textbox-question.vue

@@ -19,6 +19,7 @@
 			:validationRules="validationRules"
 			@change="handleChange"
 			@blur="handleChange"
+			:maxlength="maxLength ? maxLength : '999'"
 		/>
 		<div v-show="errorMessage" class="row my-2 form-test-error">
 			<span class="d-inline-flex mt-0" role="alert">{{ errorMessage }}</span>
@@ -55,6 +56,7 @@ export default {
 		validationRules: String,
 		semiAggressiveValidation: Boolean,
 		cmsWidgetName: String,
+		maxLength: String,
 	},
 	setup(props) {
 		const propsClone = Object.assign({}, props);

src/layouts/license-plate-lookup/license-plate-lookup.vue

@@ -24,7 +24,7 @@
               v-model="registrationZip"
               inputId="zip"
               mask="#####"
-              validationRules="zip-required"
+              validationRules="zip-required|zip-format"
             />
           </div>
         </div>
@@ -54,6 +54,7 @@
               v-model="serviceZip"
               inputId="serviceZip"
               validationRules="zip-required|zip-format"
+              semiAggressiveValidation
             />
           </div>
         </div>
@@ -87,56 +88,25 @@ import alert from "@/ux-components/alert/alert";
 import textboxQuestion from "@/common-components/textbox-question/textbox-question";
 import loadingModal from '@/common-components/loading-modal/loading-modal.vue';
 // Supporting files
-import {
+import { fetchCmsContentForPage } from "@/helpers/cms-content-helper";
-  fetchCmsContentForPage
+import { settleAllPromises } from "@/helpers/layout-helper";
-} from "@/helpers/cms-content-helper";
-import {
-   settleAllPromises
-} from "@/helpers/layout-helper";
 import store from "@/store";
 import baseMixin from "@/mixins/base-mixin.js";
-import {
+import { storeActions } from "@/constants/store-actions";
-   storeActions
+import { storeMutations } from "@/constants/store-mutations";
-} from "@/constants/store-actions";
+import { errorMessages } from "@/constants/error-messages";
-import {
+import { navigateAfterSaveToHeritageFunnel } from "@/helpers/heritage-integration/navigation-helper";
-   storeMutations
+import { getDamageString, isGlassAvailableForCarId, } from "@/helpers/damage-helper";
-} from "@/constants/store-mutations";
+import { required, regex, } from "@/helpers/validation-rules";
-import {
+import { Form, defineRule, } from "vee-validate";
-   errorMessages
-} from "@/constants/error-messages";
-import {
-  navigateAfterSaveToHeritageFunnel
-} from "@/helpers/heritage-integration/navigation-helper";
-import {
-  getDamageString,
-  isGlassAvailableForCarId,
-} from "@/helpers/damage-helper";
-import {
-   required,
-    regex,
-} from "@/helpers/validation-rules";
-import {
-   Form,
-    defineRule,
-} from "vee-validate";
 
 // DEFINE VALIDATION RULES
-defineRule(
+defineRule("license-plate-required", required(errorMessages.LICENSE_PLATE_REQUIRED));
-  "license-plate-required",
-  required(errorMessages.LICENSE_PLATE_REQUIRED)
-);
 defineRule("zip-required", required(errorMessages.REGISTRATION_ZIP_REQUIRED));
-defineRule(
+defineRule("zip-format", regex(/(^\d{5}$)|(^\d{5}-\d{4}$)/, errorMessages.SERVICE_ZIP_FORMAT));
-  "email-address-required",
+defineRule("email-address-required", required(errorMessages.EMAIL_ADDRESS_REQUIRED));
-  required(errorMessages.EMAIL_ADDRESS_REQUIRED)
+defineRule("email-address-format", regex(/^([a-zA-Z0-9_\-.+]+)@([a-zA-Z0-9_\-.]+)\.([a-zA-Z]{2,})$/, errorMessages.EMAIL_ADDRESS_FORMAT));
-);
+
-defineRule(
-  "email-address-format",
-  regex(
-    /^([a-zA-Z0-9_\-.+]+)@([a-zA-Z0-9_\-.]+)\.([a-zA-Z]{2,})$/,
-    errorMessages.EMAIL_ADDRESS_FORMAT
-  )
-);
 export default {
   name: "license-plate-lookup",
   async beforeRouteEnter(to, from, next) {

src/layouts/vin-lookup/vin-lookup.vue

@@ -24,6 +24,7 @@
               disableAutoFill
               validationRules="vin-required|vin-format"
               :isDisabled="isVinFieldReadOnly"
+              maxLength="17"
             />
           </div>
         </div>
@@ -41,7 +42,7 @@
               mask="#####"
               isRequired
               disableAutoFill
-              validationRules="zip-required"
+              validationRules="zip-required|zip-format"
             />
           </div>
         </div>
@@ -141,29 +142,12 @@ import { Form, defineRule } from "vee-validate";
 import { navigateAfterSaveToHeritageFunnel } from "@/helpers/heritage-integration/navigation-helper";
 
 // DEFINE VALIDATION RULES
-defineRule(
+defineRule("zip-required", required(errorMessages.SERVICE_ZIP_REQUIRED));
-  "zip-required",
+defineRule("zip-format", regex(/(^\d{5}$)|(^\d{5}-\d{4}$)/, errorMessages.SERVICE_ZIP_FORMAT));
-  required(errorMessages.SERVICE_ZIP_REQUIRED)
+defineRule("email-address-required", required(errorMessages.EMAIL_ADDRESS_REQUIRED));
-);
+defineRule("email-address-format", regex(/^([a-zA-Z0-9_\-.+]+)@([a-zA-Z0-9_\-.]+)\.([a-zA-Z]{2,})$/, errorMessages.EMAIL_ADDRESS_FORMAT));
-defineRule(
+defineRule("vin-required", required(errorMessages.VIN_REQUIRED));
-  "email-address-required",
+defineRule("vin-format", regex(/^[A-HJ-NPR-Z0-9]{17}$/, errorMessages.VIN_FORMAT));
-  required(errorMessages.EMAIL_ADDRESS_REQUIRED)
-);
-defineRule(
-  "email-address-format",
-  regex(
-    /^([a-zA-Z0-9_\-.+]+)@([a-zA-Z0-9_\-.]+)\.([a-zA-Z]{2,})$/,
-    errorMessages.EMAIL_ADDRESS_FORMAT
-  )
-);
-defineRule(
-  "vin-required",
-  required(errorMessages.VIN_REQUIRED)
-);
-defineRule(
-  "vin-format",
-  regex(/^[a-hA-Hj-nJ-NpPr-zR-Z0-9]{17}$/, errorMessages.VIN_FORMAT)
-);
 
 export default {
   name: "vin-lookup",