From 4dd04831745bd8a25c2ac316ee572f30fe2a4b36 Mon Sep 17 00:00:00 2001
From: Christopher Bisom <cbisom@flexion.us>
Date: Fri, 13 Sep 2024 15:04:14 -0400
Subject: [PATCH 01/33] 10074-bug: initial work

---
 .../actions/setValidationAlertErrorsAction.ts |  1 +
 .../src/presenter/computeds/alertHelper.ts    |  1 -
 .../views/CaseDetailEdit/PartyInformation.tsx |  4 +++-
 web-client/src/views/ErrorNotification.tsx    |  4 ++--
 web-client/src/views/StartCase/Address.tsx    | 24 +++++++++----------
 .../src/views/StartCase/ContactPrimary.tsx    | 16 ++++---------
 .../src/views/StartCase/ContactSecondary.tsx  | 14 +++--------
 web-client/src/views/StartCase/Country.tsx    | 18 +++++++-------
 .../views/StartCase/InternationalAddress.tsx  | 17 +++++++------
 .../src/views/StartCase/StartCaseStep3.tsx    |  2 +-
 .../StartCaseInternal/CaseInformation.tsx     | 18 ++++++++------
 .../src/views/StartCaseInternal/Parties.tsx   |  6 +++--
 12 files changed, 59 insertions(+), 66 deletions(-)

diff --git a/web-client/src/presenter/actions/setValidationAlertErrorsAction.ts b/web-client/src/presenter/actions/setValidationAlertErrorsAction.ts
index a4ae8199ac5..3dd3263d756 100644
--- a/web-client/src/presenter/actions/setValidationAlertErrorsAction.ts
+++ b/web-client/src/presenter/actions/setValidationAlertErrorsAction.ts
@@ -86,6 +86,7 @@ export const setValidationAlertErrorsAction = ({
           }
         }),
     ),
+    preventAutoScroll: props.preventAutoScroll || false,
     title: 'Please correct the following errors on the page:',
   };
   store.set(state.alertError, alertError);
diff --git a/web-client/src/presenter/computeds/alertHelper.ts b/web-client/src/presenter/computeds/alertHelper.ts
index 7f98c6267c9..9fab5e99a36 100644
--- a/web-client/src/presenter/computeds/alertHelper.ts
+++ b/web-client/src/presenter/computeds/alertHelper.ts
@@ -15,7 +15,6 @@ export const alertHelper = (get: Get): any => {
 
   return {
     messagesDeduped: uniq(alertError.messages).filter(Boolean),
-    preventAutoScroll: false,
     responseCode: alertError.responseCode,
     showErrorAlert:
       !!alertError.title || !!alertError.message || !!alertError.messages,
diff --git a/web-client/src/views/CaseDetailEdit/PartyInformation.tsx b/web-client/src/views/CaseDetailEdit/PartyInformation.tsx
index f6fcd70dbea..b498cdf7995 100644
--- a/web-client/src/views/CaseDetailEdit/PartyInformation.tsx
+++ b/web-client/src/views/CaseDetailEdit/PartyInformation.tsx
@@ -11,12 +11,14 @@ export const PartyInformation = connect(
     form: state.form,
     updateCasePartyTypeSequence: sequences.updateCasePartyTypeSequence,
     updateFormValueSequence: sequences.updateFormValueSequence,
+    validateCaseDetailSequence: sequences.validateCaseDetailSequence,
   },
   function PartyInformation({
     caseDetailEditHelper,
     form,
     updateCasePartyTypeSequence,
     updateFormValueSequence,
+    validateCaseDetailSequence,
   }) {
     return (
       <div className="blue-container document-detail-one-third">
@@ -87,7 +89,7 @@ export const PartyInformation = connect(
               showPrimaryContact={caseDetailEditHelper.showPrimaryContact}
               showSecondaryContact={caseDetailEditHelper.showSecondaryContact}
               useSameAsPrimary={true}
-              onBlur="validateCaseDetailSequence"
+              onBlur={validateCaseDetailSequence}
               onChange="updateFormValueAndCaseCaptionSequence"
             />
           </div>
diff --git a/web-client/src/views/ErrorNotification.tsx b/web-client/src/views/ErrorNotification.tsx
index c7486c5ed63..6e80076d0fa 100644
--- a/web-client/src/views/ErrorNotification.tsx
+++ b/web-client/src/views/ErrorNotification.tsx
@@ -15,9 +15,9 @@ export const ErrorNotification = connect(
     alertError?: {
       title?: string;
       message?: string;
+      preventAutoScroll?: boolean;
     };
     alertHelper: {
-      preventAutoScroll?: boolean;
       showErrorAlert?: boolean;
       showSingleMessage?: boolean;
       showMultipleMessages?: boolean;
@@ -29,7 +29,7 @@ export const ErrorNotification = connect(
 
     useEffect(() => {
       const notification = notificationRef.current;
-      if (notification && !alertHelper.preventAutoScroll) {
+      if (notification && !alertError?.preventAutoScroll) {
         window.scrollTo(0, 0);
       }
     });
diff --git a/web-client/src/views/StartCase/Address.tsx b/web-client/src/views/StartCase/Address.tsx
index 78735ecf2ae..bb96cff0b74 100644
--- a/web-client/src/views/StartCase/Address.tsx
+++ b/web-client/src/views/StartCase/Address.tsx
@@ -11,7 +11,7 @@ import classNames from 'classnames';
 export const Address = connect(
   {
     data: state[props.bind],
-    onBlurSequence: sequences[props.onBlur],
+    onBlur: props.onBlur,
     registerRef: props.registerRef,
     type: props.type,
     updateFormValueAndSecondaryContactInfoSequence: sequences[props.onChange],
@@ -20,7 +20,7 @@ export const Address = connect(
   },
   function Address({
     data,
-    onBlurSequence,
+    onBlur,
     registerRef,
     type,
     updateFormValueAndSecondaryContactInfoSequence,
@@ -38,7 +38,7 @@ export const Address = connect(
               useFullStateName
               data={data}
               handleBlur={() =>
-                onBlurSequence({
+                onBlur({
                   validationKey: [type, 'state'],
                 })
               }
@@ -46,7 +46,7 @@ export const Address = connect(
               refProp={registerRef && registerRef(`${type}.state`)}
               type={type}
               onChangeValidationSequence={() =>
-                onBlurSequence({
+                onBlur({
                   validationKey: [type, 'state'],
                 })
               }
@@ -70,7 +70,7 @@ export const Address = connect(
               type="text"
               value={data[type].postalCode || ''}
               onBlur={() => {
-                onBlurSequence({
+                onBlur({
                   validationKey: [type, 'postalCode'],
                 });
               }}
@@ -109,7 +109,7 @@ export const Address = connect(
                   className="max-width-180"
                   data={data}
                   handleBlur={() =>
-                    onBlurSequence({
+                    onBlur({
                       validationKey: [type, 'state'],
                     })
                   }
@@ -117,7 +117,7 @@ export const Address = connect(
                   refProp={registerRef && registerRef(`${type}.state`)}
                   type={type}
                   onChangeValidationSequence={() =>
-                    onBlurSequence({
+                    onBlur({
                       validationKey: [type, 'state'],
                     })
                   }
@@ -152,7 +152,7 @@ export const Address = connect(
                   type="text"
                   value={data[type].postalCode || ''}
                   onBlur={() => {
-                    onBlurSequence({
+                    onBlur({
                       validationKey: [type, 'postalCode'],
                     });
                   }}
@@ -198,7 +198,7 @@ export const Address = connect(
             type="text"
             value={data[type].address1 || ''}
             onBlur={() => {
-              onBlurSequence({
+              onBlur({
                 validationKey: [type, 'address1'],
               });
             }}
@@ -222,7 +222,7 @@ export const Address = connect(
             type="text"
             value={data[type].address2 || ''}
             onBlur={() => {
-              onBlurSequence({
+              onBlur({
                 validationKey: [type, 'address2'],
               });
             }}
@@ -246,7 +246,7 @@ export const Address = connect(
             type="text"
             value={data[type].address3 || ''}
             onBlur={() => {
-              onBlurSequence({
+              onBlur({
                 validationKey: [type, 'address3'],
               });
             }}
@@ -275,7 +275,7 @@ export const Address = connect(
             type="text"
             value={data[type].city || ''}
             onBlur={() => {
-              onBlurSequence({
+              onBlur({
                 validationKey: [type, 'city'],
               });
             }}
diff --git a/web-client/src/views/StartCase/ContactPrimary.tsx b/web-client/src/views/StartCase/ContactPrimary.tsx
index 16b80966a18..2465a447cf4 100644
--- a/web-client/src/views/StartCase/ContactPrimary.tsx
+++ b/web-client/src/views/StartCase/ContactPrimary.tsx
@@ -13,7 +13,7 @@ import React from 'react';
 const props = cerebralProps as unknown as {
   contactsHelper: string;
   bind: string;
-  onBlur: string;
+  onBlur: () => void;
   onChange: string;
   parentView: string;
 };
@@ -25,7 +25,6 @@ export const ContactPrimary = connect(
     contactsHelper: state[props.contactsHelper],
     data: state[props.bind],
     onBlur: props.onBlur,
-    onBlurSequence: sequences[props.onBlur],
     onChange: props.onChange,
     onChangeSequence: sequences[props.onChange],
     parentView: props.parentView,
@@ -39,7 +38,6 @@ export const ContactPrimary = connect(
     contactsHelper,
     data,
     onBlur,
-    onBlurSequence,
     onChange,
     onChangeSequence,
     parentView,
@@ -139,9 +137,7 @@ export const ContactPrimary = connect(
           name="contactPrimary.inCareOf"
           type="text"
           value={data.contactPrimary.inCareOf || ''}
-          onBlur={() => {
-            onBlurSequence();
-          }}
+          onBlur={onBlur}
           onChange={e => {
             onChangeSequence({
               key: e.target.name,
@@ -179,9 +175,7 @@ export const ContactPrimary = connect(
               name="contactPrimary.name"
               type="text"
               value={data.contactPrimary.name || ''}
-              onBlur={() => {
-                onBlurSequence();
-              }}
+              onBlur={onBlur}
               onChange={e => {
                 onChangeSequence({
                   key: e.target.name,
@@ -253,9 +247,7 @@ export const ContactPrimary = connect(
               name="contactPrimary.phone"
               type="text"
               value={data.contactPrimary.phone || ''}
-              onBlur={() => {
-                onBlurSequence();
-              }}
+              onBlur={onBlur}
               onChange={e => {
                 updateFormValueAndSecondaryContactInfoSequence({
                   key: e.target.name,
diff --git a/web-client/src/views/StartCase/ContactSecondary.tsx b/web-client/src/views/StartCase/ContactSecondary.tsx
index 560eaa58a42..cb3113833b1 100644
--- a/web-client/src/views/StartCase/ContactSecondary.tsx
+++ b/web-client/src/views/StartCase/ContactSecondary.tsx
@@ -17,7 +17,6 @@ export const ContactSecondary = connect(
     contactsHelper: state[props.contactsHelper],
     data: state[props.bind],
     onBlur: props.onBlur,
-    onBlurSequence: sequences[props.onBlur],
     onChange: props.onChange,
     onChangeSequence: sequences[props.onChange],
     parentView: props.parentView,
@@ -33,7 +32,6 @@ export const ContactSecondary = connect(
     contactsHelper,
     data,
     onBlur,
-    onBlurSequence,
     onChange,
     onChangeSequence,
     parentView,
@@ -70,9 +68,7 @@ export const ContactSecondary = connect(
               name="contactSecondary.name"
               type="text"
               value={data.contactSecondary.name || ''}
-              onBlur={() => {
-                onBlurSequence();
-              }}
+              onBlur={onBlur}
               onChange={e => {
                 onChangeSequence({
                   key: e.target.name,
@@ -137,9 +133,7 @@ export const ContactSecondary = connect(
                 name="contactSecondary.inCareOf"
                 type="text"
                 value={data.contactSecondary.inCareOf || ''}
-                onBlur={() => {
-                  onBlurSequence();
-                }}
+                onBlur={onBlur}
                 onChange={e => {
                   onChangeSequence({
                     key: e.target.name,
@@ -203,9 +197,7 @@ export const ContactSecondary = connect(
                 name="contactSecondary.phone"
                 type="text"
                 value={data.contactSecondary.phone || ''}
-                onBlur={() => {
-                  onBlurSequence();
-                }}
+                onBlur={onBlur}
                 onChange={e => {
                   onChangeSequence({
                     key: e.target.name,
diff --git a/web-client/src/views/StartCase/Country.tsx b/web-client/src/views/StartCase/Country.tsx
index 32a5ed3036e..df4b99b9ce4 100644
--- a/web-client/src/views/StartCase/Country.tsx
+++ b/web-client/src/views/StartCase/Country.tsx
@@ -9,7 +9,7 @@ export const Country = connect(
   {
     constants: state.constants,
     data: state[props.bind],
-    onBlurSequence: sequences[props.onBlur],
+    onBlur: props.onBlur,
     onChangeCountryType: sequences[props.onChangeCountryType],
     registerRef: props.registerRef,
     type: props.type,
@@ -19,7 +19,7 @@ export const Country = connect(
   function Country({
     constants,
     data,
-    onBlurSequence,
+    onBlur,
     onChangeCountryType,
     registerRef,
     type,
@@ -50,8 +50,10 @@ export const Country = connect(
                   type,
                   value: e.target.value,
                 });
-                if (onBlurSequence) {
-                  onBlurSequence({ validationKey: [type, 'countryType'] });
+                if (onBlur) {
+                  onBlur({
+                    validationKey: [type, 'countryType'],
+                  });
                 }
               }}
             />
@@ -79,8 +81,8 @@ export const Country = connect(
                   type,
                   value: e.target.value,
                 });
-                if (onBlurSequence) {
-                  onBlurSequence({
+                if (onBlur) {
+                  onBlur({
                     validationKey: [type, 'countryType'],
                   });
                 }
@@ -114,8 +116,8 @@ export const Country = connect(
               type="text"
               value={data[type].country || ''}
               onBlur={() => {
-                if (onBlurSequence) {
-                  onBlurSequence({
+                if (onBlur) {
+                  onBlur({
                     validationKey: [type, 'country'],
                   });
                 }
diff --git a/web-client/src/views/StartCase/InternationalAddress.tsx b/web-client/src/views/StartCase/InternationalAddress.tsx
index f931dd25445..2e4307dc4f7 100644
--- a/web-client/src/views/StartCase/InternationalAddress.tsx
+++ b/web-client/src/views/StartCase/InternationalAddress.tsx
@@ -7,7 +7,7 @@ import React from 'react';
 
 const props = cerebralProps as unknown as {
   bind: string;
-  onBlur: string;
+  onBlur: any;
   type: string;
   onChange: string;
   registerRef: (param: string) => void;
@@ -17,7 +17,6 @@ export const InternationalAddress = connect(
   {
     data: state[props.bind],
     onBlur: props.onBlur,
-    onBlurSequence: sequences[props.onBlur],
     registerRef: props.registerRef,
     type: props.type,
     updateFormValueSequence: sequences[props.onChange],
@@ -25,7 +24,7 @@ export const InternationalAddress = connect(
   },
   function InternationalAddress({
     data,
-    onBlurSequence,
+    onBlur,
     registerRef,
     type,
     updateFormValueSequence,
@@ -50,7 +49,7 @@ export const InternationalAddress = connect(
             type="text"
             value={data[type].address1 || ''}
             onBlur={() => {
-              onBlurSequence({
+              onBlur({
                 validationKey: [type, 'address1'],
               });
             }}
@@ -75,7 +74,7 @@ export const InternationalAddress = connect(
             type="text"
             value={data[type].address2 || ''}
             onBlur={() => {
-              onBlurSequence({
+              onBlur({
                 validationKey: [type, 'address2'],
               });
             }}
@@ -100,7 +99,7 @@ export const InternationalAddress = connect(
             type="text"
             value={data[type].address3 || ''}
             onBlur={() => {
-              onBlurSequence({
+              onBlur({
                 validationKey: [type, 'address3'],
               });
             }}
@@ -129,7 +128,7 @@ export const InternationalAddress = connect(
             type="text"
             value={data[type].state || ''}
             onBlur={() => {
-              onBlurSequence({
+              onBlur({
                 validationKey: [type, 'state'],
               });
             }}
@@ -158,7 +157,7 @@ export const InternationalAddress = connect(
             type="text"
             value={data[type].city || ''}
             onBlur={() => {
-              onBlurSequence({
+              onBlur({
                 validationKey: [type, 'city'],
               });
             }}
@@ -187,7 +186,7 @@ export const InternationalAddress = connect(
             type="text"
             value={data[type].postalCode || ''}
             onBlur={() => {
-              onBlurSequence({
+              onBlur({
                 validationKey: [type, 'postalCode'],
               });
             }}
diff --git a/web-client/src/views/StartCase/StartCaseStep3.tsx b/web-client/src/views/StartCase/StartCaseStep3.tsx
index 3ac12d536e5..5c538c0226c 100644
--- a/web-client/src/views/StartCase/StartCaseStep3.tsx
+++ b/web-client/src/views/StartCase/StartCaseStep3.tsx
@@ -349,7 +349,7 @@ export const StartCaseStep3 = connect(
           showPrimaryContact={startCaseHelper.showPrimaryContact}
           showSecondaryContact={startCaseHelper.showSecondaryContact}
           useSameAsPrimary={true}
-          onBlur="validateStartCaseWizardSequence"
+          onBlur={validateStartCaseWizardSequence}
           onChange="updateFormValueSequence"
         />
 
diff --git a/web-client/src/views/StartCaseInternal/CaseInformation.tsx b/web-client/src/views/StartCaseInternal/CaseInformation.tsx
index ac40848fe84..e7f25e27851 100644
--- a/web-client/src/views/StartCaseInternal/CaseInformation.tsx
+++ b/web-client/src/views/StartCaseInternal/CaseInformation.tsx
@@ -53,7 +53,7 @@ export const CaseInformation = connect(
               toFormat: DATE_FORMATS.ISO,
               value: e.target.value,
             });
-            validatePetitionFromPaperSequence();
+            validatePetitionFromPaperSequence({ preventAutoScroll: true });
           }}
         />
         <FormGroup errorText={validationErrors.mailingDate}>
@@ -66,7 +66,9 @@ export const CaseInformation = connect(
             maxLength="25"
             name="mailingDate"
             value={form.mailingDate || ''}
-            onBlur={() => validatePetitionFromPaperSequence()}
+            onBlur={() =>
+              validatePetitionFromPaperSequence({ preventAutoScroll: true })
+            }
             onChange={e => {
               updateFormValueSequence({
                 key: e.target.name,
@@ -85,7 +87,7 @@ export const CaseInformation = connect(
             name="caseCaption"
             value={form.caseCaption}
             onBlur={() => {
-              validatePetitionFromPaperSequence();
+              validatePetitionFromPaperSequence({ preventAutoScroll: true });
             }}
             onChange={e => {
               updateFormValueSequence({
@@ -107,7 +109,7 @@ export const CaseInformation = connect(
               value: e.target.value,
             });
             clearPreferredTrialCitySequence();
-            validatePetitionFromPaperSequence();
+            validatePetitionFromPaperSequence({ preventAutoScroll: true });
           }}
         />
         <FormGroup>
@@ -148,7 +150,7 @@ export const CaseInformation = connect(
               key: e.target.name,
               value: e.target.value || null,
             });
-            validatePetitionFromPaperSequence();
+            validatePetitionFromPaperSequence({ preventAutoScroll: true });
           }}
         />
         <FormGroup errorText={validationErrors['object.missing']}>
@@ -165,7 +167,7 @@ export const CaseInformation = connect(
                   key: e.target.name,
                   value: e.target.checked,
                 });
-                validatePetitionFromPaperSequence();
+                validatePetitionFromPaperSequence({ preventAutoScroll: true });
               }}
             />
             <label
@@ -180,7 +182,9 @@ export const CaseInformation = connect(
         <PetitionPaymentForm
           bind="form"
           updateSequence={updatePetitionPaymentFormValueSequence}
-          validateSequence={validatePetitionFromPaperSequence}
+          validateSequence={() =>
+            validatePetitionFromPaperSequence({ preventAutoScroll: true })
+          }
           validationErrorsBind="validationErrors"
         />
         {startCaseInternalHelper.showOrderForFilingFee && (
diff --git a/web-client/src/views/StartCaseInternal/Parties.tsx b/web-client/src/views/StartCaseInternal/Parties.tsx
index c4cd9b98c2a..ac30f1a3726 100644
--- a/web-client/src/views/StartCaseInternal/Parties.tsx
+++ b/web-client/src/views/StartCaseInternal/Parties.tsx
@@ -40,7 +40,7 @@ export const Parties = connect(
                 key: e.target.name,
                 value: e.target.value,
               });
-              validatePetitionFromPaperSequence();
+              validatePetitionFromPaperSequence({ preventAutoScroll: true });
             }}
           >
             <option value="">- Select -</option>
@@ -92,7 +92,9 @@ export const Parties = connect(
                 startCaseInternalHelper.showSecondaryContact
               }
               useSameAsPrimary={true}
-              onBlur="validatePetitionFromPaperSequence"
+              onBlur={() =>
+                validatePetitionFromPaperSequence({ preventAutoScroll: true })
+              }
               onChange="updateFormValueAndCaseCaptionSequence"
             />
           </div>

From d63f3bc713a75c3993d8494487d2a444b635a5b0 Mon Sep 17 00:00:00 2001
From: Christopher Bisom <cbisom@flexion.us>
Date: Fri, 13 Sep 2024 15:14:30 -0400
Subject: [PATCH 02/33] 10074-bug: fix more onBlurs

---
 web-client/src/views/StartCase/ContactPrimary.tsx | 1 +
 1 file changed, 1 insertion(+)

diff --git a/web-client/src/views/StartCase/ContactPrimary.tsx b/web-client/src/views/StartCase/ContactPrimary.tsx
index 2465a447cf4..03c968eaec3 100644
--- a/web-client/src/views/StartCase/ContactPrimary.tsx
+++ b/web-client/src/views/StartCase/ContactPrimary.tsx
@@ -69,6 +69,7 @@ export const ContactPrimary = connect(
           name="contactPrimary.secondaryName"
           type="text"
           value={data.contactPrimary.secondaryName || ''}
+          onBlur={onBlur}
           onChange={e => {
             onChangeSequence({
               key: e.target.name,

From 285a98e4f511f40ac04b9ed8da7b612ae86d5887 Mon Sep 17 00:00:00 2001
From: Christopher Bisom <cbisom@flexion.us>
Date: Fri, 13 Sep 2024 15:37:25 -0400
Subject: [PATCH 03/33] 10074-bug: trying to standardize onBlurs in the form

---
 .../src/views/CaseDetail/EditCaseDetails.tsx     |  2 +-
 .../src/views/CaseDetailEdit/CaseDetailEdit.tsx  |  4 +++-
 web-client/src/views/IRSNotice.tsx               |  9 ++++-----
 .../src/views/StartCase/CaseTypeSelect.tsx       |  6 +++---
 .../src/views/StartCase/StartCaseStep2.tsx       |  4 ++--
 .../StartCaseInternal/StartCaseInternal.tsx      | 16 ++++++++++++++--
 .../views/StartCaseInternal/StatisticsForm.tsx   | 10 ++++++++--
 7 files changed, 35 insertions(+), 16 deletions(-)

diff --git a/web-client/src/views/CaseDetail/EditCaseDetails.tsx b/web-client/src/views/CaseDetail/EditCaseDetails.tsx
index 53ecf2f504e..d3dc5af3adb 100644
--- a/web-client/src/views/CaseDetail/EditCaseDetails.tsx
+++ b/web-client/src/views/CaseDetail/EditCaseDetails.tsx
@@ -45,7 +45,7 @@ export const EditCaseDetails = connect(
               <h4 className="margin-bottom-2">IRS Notice/Case</h4>
               <IRSNotice
                 shouldStartWithBlankStatistic={false}
-                validationName="validateCaseDetailsSequence"
+                validationFunction={validateCaseDetailsSequence}
               />
 
               <ProcedureType
diff --git a/web-client/src/views/CaseDetailEdit/CaseDetailEdit.tsx b/web-client/src/views/CaseDetailEdit/CaseDetailEdit.tsx
index 292a39ae771..cbc76ab7e98 100644
--- a/web-client/src/views/CaseDetailEdit/CaseDetailEdit.tsx
+++ b/web-client/src/views/CaseDetailEdit/CaseDetailEdit.tsx
@@ -14,11 +14,13 @@ export const CaseDetailEdit = connect(
     navigateBackSequence: sequences.navigateBackSequence,
     saveSavedCaseForLaterSequence: sequences.saveSavedCaseForLaterSequence,
     screenMetadata: state.screenMetadata,
+    validateCaseDetailSequence: sequences.validateCaseDetailSequence,
   },
   function CaseDetailEdit({
     navigateBackSequence,
     saveSavedCaseForLaterSequence,
     screenMetadata,
+    validateCaseDetailSequence,
   }) {
     return (
       <div noValidate id="case-edit-form" role="form">
@@ -46,7 +48,7 @@ export const CaseDetailEdit = connect(
             title="IRS Notice"
           >
             <div className="blue-container">
-              <IRSNotice validationName="validateCaseDetailSequence" />
+              <IRSNotice validationFunction={validateCaseDetailSequence} />
             </div>
           </Tab>
         </Tabs>
diff --git a/web-client/src/views/IRSNotice.tsx b/web-client/src/views/IRSNotice.tsx
index b64e24f720e..cf073285d75 100644
--- a/web-client/src/views/IRSNotice.tsx
+++ b/web-client/src/views/IRSNotice.tsx
@@ -21,8 +21,8 @@ export const IRSNotice = connect(
     showModal: state.modal.showModal,
     statisticsFormHelper: state.statisticsFormHelper,
     updateFormValueSequence: sequences.updateFormValueSequence,
-    validation: sequences[props.validationName],
     validationErrors: state.validationErrors,
+    validationFunction: props.validationFunction,
   },
   function IRSNotice({
     caseDetailEditHelper,
@@ -35,9 +35,8 @@ export const IRSNotice = connect(
     showModal,
     statisticsFormHelper,
     updateFormValueSequence,
-    validation,
     validationErrors,
-    validationName,
+    validationFunction,
   }) {
     const renderIrsNoticeRadios = () => {
       return (
@@ -120,7 +119,7 @@ export const IRSNotice = connect(
               toFormat: constants.DATE_FORMATS.ISO,
               value: e.target.value,
             });
-            validation();
+            validationFunction({ preventAutoScroll: true });
           }}
         />
       );
@@ -134,7 +133,7 @@ export const IRSNotice = connect(
           allowDefaultOption={true}
           caseTypes={constants.CASE_TYPES}
           legend="Type of case"
-          validation={validationName}
+          validationFunction={validationFunction}
           value={form.caseType}
           onChange="updateFormValueSequence"
           onChangePreValidation="refreshStatisticsSequence"
diff --git a/web-client/src/views/StartCase/CaseTypeSelect.tsx b/web-client/src/views/StartCase/CaseTypeSelect.tsx
index c352a6e091c..196d7b2dc3f 100644
--- a/web-client/src/views/StartCase/CaseTypeSelect.tsx
+++ b/web-client/src/views/StartCase/CaseTypeSelect.tsx
@@ -19,9 +19,9 @@ export const CaseTypeSelect = connect(
     onChangePreValidation: sequences[props.onChangePreValidation],
     rawOnChange: props.onChange,
     refProp: props.refProp,
-    validation: sequences[props.validation],
     validationError: props.validationError,
     validationErrors: state.validationErrors,
+    validationFunction: props.validationFunction,
     value: props.value,
   },
   function CaseTypeSelect({
@@ -37,9 +37,9 @@ export const CaseTypeSelect = connect(
     onChangePreValidation,
     rawOnChange,
     refProp,
-    validation,
     validationError,
     validationErrors,
+    validationFunction,
     value,
   }) {
     return (
@@ -69,7 +69,7 @@ export const CaseTypeSelect = connect(
                   value: e.target.value,
                 });
                 if (onChangePreValidation) onChangePreValidation();
-                if (validation) validation();
+                if (validationFunction) validationFunction();
               }}
             >
               {allowDefaultOption && <option value="">-- Select --</option>}
diff --git a/web-client/src/views/StartCase/StartCaseStep2.tsx b/web-client/src/views/StartCase/StartCaseStep2.tsx
index aba07157a70..72026589532 100644
--- a/web-client/src/views/StartCase/StartCaseStep2.tsx
+++ b/web-client/src/views/StartCase/StartCaseStep2.tsx
@@ -145,7 +145,7 @@ export const StartCaseStep2 = connect(
                   allowDefaultOption={true}
                   caseTypes={caseTypeDescriptionHelper.caseTypes}
                   legend="Type of notice / case"
-                  validation="validateStartCaseWizardSequence"
+                  validationFunction={validateStartCaseWizardSequence}
                   value={form.caseType}
                   onChange="updateFormValueSequence"
                 />
@@ -193,7 +193,7 @@ export const StartCaseStep2 = connect(
                 className="margin-bottom-0"
                 legend="Which topic most closely matches your complaint with the
                 IRS?"
-                validation="validateStartCaseWizardSequence"
+                validationFunction={validateStartCaseWizardSequence}
                 value={form.caseType}
                 onChange="updateFormValueSequence"
               />
diff --git a/web-client/src/views/StartCaseInternal/StartCaseInternal.tsx b/web-client/src/views/StartCaseInternal/StartCaseInternal.tsx
index 8038dd93aca..812a57fb005 100644
--- a/web-client/src/views/StartCaseInternal/StartCaseInternal.tsx
+++ b/web-client/src/views/StartCaseInternal/StartCaseInternal.tsx
@@ -23,6 +23,7 @@ export const StartCaseInternal = connect(
     showModal: state.modal.showModal,
     startCaseHelper: state.startCaseHelper,
     submitPetitionFromPaperSequence: sequences.submitPetitionFromPaperSequence,
+    validateCaseDetailSequence: sequences.validateCaseDetailSequence,
     validatePetitionFromPaperSequence:
       sequences.validatePetitionFromPaperSequence,
   },
@@ -33,6 +34,7 @@ export const StartCaseInternal = connect(
     showModal,
     startCaseHelper,
     submitPetitionFromPaperSequence,
+    validateCaseDetailSequence,
     validatePetitionFromPaperSequence,
   }) {
     return (
@@ -75,7 +77,13 @@ export const StartCaseInternal = connect(
                     title="IRS Notice"
                   >
                     <div className="blue-container">
-                      <IRSNotice validationName="validateCaseDetailSequence" />
+                      <IRSNotice
+                        validationFunction={() =>
+                          validateCaseDetailSequence({
+                            preventAutoScroll: true,
+                          })
+                        }
+                      />
                     </div>
                   </Tab>
                 </Tabs>
@@ -85,7 +93,11 @@ export const StartCaseInternal = connect(
                   documentTabs={startCaseHelper.documentTabs}
                   documentType={documentSelectedForScan}
                   title="Add Document(s)"
-                  validateSequence={validatePetitionFromPaperSequence}
+                  validateSequence={() =>
+                    validatePetitionFromPaperSequence({
+                      preventAutoScroll: true,
+                    })
+                  }
                 />
               </div>
             </div>
diff --git a/web-client/src/views/StartCaseInternal/StatisticsForm.tsx b/web-client/src/views/StartCaseInternal/StatisticsForm.tsx
index f9e1a099edf..1cc91db8c2b 100644
--- a/web-client/src/views/StartCaseInternal/StatisticsForm.tsx
+++ b/web-client/src/views/StartCaseInternal/StatisticsForm.tsx
@@ -49,7 +49,9 @@ export const StatisticsForm = connect(
           id={`deficiency-amount-${index}`}
           name={`statistics.${index}.irsDeficiencyAmount`}
           value={form.statistics[index].irsDeficiencyAmount || ''}
-          onBlur={() => validatePetitionFromPaperSequence()}
+          onBlur={() =>
+            validatePetitionFromPaperSequence({ preventAutoScroll: true })
+          }
           onValueChange={values => {
             updateStatisticsFormValueSequence({
               key: `statistics.${index}.irsDeficiencyAmount`,
@@ -132,7 +134,11 @@ export const StatisticsForm = connect(
                     name={`statistics.${index}.year`}
                     placeholder="YYYY"
                     value={form.statistics[index].year || ''}
-                    onBlur={() => validatePetitionFromPaperSequence()}
+                    onBlur={() =>
+                      validatePetitionFromPaperSequence({
+                        preventAutoScroll: true,
+                      })
+                    }
                     onChange={e => {
                       updateStatisticsFormValueSequence({
                         key: e.target.name,

From 19b8053de3b0cc484267d914479ae449735ae355 Mon Sep 17 00:00:00 2001
From: Christopher Bisom <cbisom@flexion.us>
Date: Mon, 16 Sep 2024 10:59:09 -0400
Subject: [PATCH 04/33] 10074-bug: fixing some missed onBlurs, adding typing to
 onBlur

---
 web-client/src/views/CaseDetailEdit/PartyInformation.tsx | 4 +++-
 web-client/src/views/StartCase/CaseTypeSelect.tsx        | 3 ++-
 web-client/src/views/StartCase/ContactPrimary.tsx        | 2 +-
 web-client/src/views/StartCase/InternationalAddress.tsx  | 2 +-
 4 files changed, 7 insertions(+), 4 deletions(-)

diff --git a/web-client/src/views/CaseDetailEdit/PartyInformation.tsx b/web-client/src/views/CaseDetailEdit/PartyInformation.tsx
index b498cdf7995..a52690ce86a 100644
--- a/web-client/src/views/CaseDetailEdit/PartyInformation.tsx
+++ b/web-client/src/views/CaseDetailEdit/PartyInformation.tsx
@@ -89,7 +89,9 @@ export const PartyInformation = connect(
               showPrimaryContact={caseDetailEditHelper.showPrimaryContact}
               showSecondaryContact={caseDetailEditHelper.showSecondaryContact}
               useSameAsPrimary={true}
-              onBlur={validateCaseDetailSequence}
+              onBlur={() =>
+                validateCaseDetailSequence({ preventAutoScroll: true })
+              }
               onChange="updateFormValueAndCaseCaptionSequence"
             />
           </div>
diff --git a/web-client/src/views/StartCase/CaseTypeSelect.tsx b/web-client/src/views/StartCase/CaseTypeSelect.tsx
index 196d7b2dc3f..560b62cae9a 100644
--- a/web-client/src/views/StartCase/CaseTypeSelect.tsx
+++ b/web-client/src/views/StartCase/CaseTypeSelect.tsx
@@ -69,7 +69,8 @@ export const CaseTypeSelect = connect(
                   value: e.target.value,
                 });
                 if (onChangePreValidation) onChangePreValidation();
-                if (validationFunction) validationFunction();
+                if (validationFunction)
+                  validationFunction({ preventAutoScroll: true });
               }}
             >
               {allowDefaultOption && <option value="">-- Select --</option>}
diff --git a/web-client/src/views/StartCase/ContactPrimary.tsx b/web-client/src/views/StartCase/ContactPrimary.tsx
index 03c968eaec3..063e1b90bc2 100644
--- a/web-client/src/views/StartCase/ContactPrimary.tsx
+++ b/web-client/src/views/StartCase/ContactPrimary.tsx
@@ -13,7 +13,7 @@ import React from 'react';
 const props = cerebralProps as unknown as {
   contactsHelper: string;
   bind: string;
-  onBlur: () => void;
+  onBlur: (args: Record<string, any>) => void;
   onChange: string;
   parentView: string;
 };
diff --git a/web-client/src/views/StartCase/InternationalAddress.tsx b/web-client/src/views/StartCase/InternationalAddress.tsx
index 2e4307dc4f7..f1081b505d2 100644
--- a/web-client/src/views/StartCase/InternationalAddress.tsx
+++ b/web-client/src/views/StartCase/InternationalAddress.tsx
@@ -7,7 +7,7 @@ import React from 'react';
 
 const props = cerebralProps as unknown as {
   bind: string;
-  onBlur: any;
+  onBlur: (args: Record<string, any>) => void;
   type: string;
   onChange: string;
   registerRef: (param: string) => void;

From 82e3b6a0a981df4e61cd5c703ffe7338e61171a3 Mon Sep 17 00:00:00 2001
From: Christopher Bisom <cbisom@flexion.us>
Date: Mon, 16 Sep 2024 15:17:16 -0400
Subject: [PATCH 05/33] 10074-bug: update even more onBlurs

---
 .../respondent-modifies-contact-info.cy.ts     |  2 +-
 .../AddPetitionerToCase.tsx                    |  3 ++-
 .../EditPetitionerInformationInternal.tsx      |  3 ++-
 .../Practitioners/PractitionerContactForm.tsx  | 13 ++++++-------
 web-client/src/views/UserContactEditForm.tsx   | 18 ++++++++++++------
 5 files changed, 23 insertions(+), 16 deletions(-)

diff --git a/cypress/local-only/tests/integration/myAccount/respondent-modifies-contact-info.cy.ts b/cypress/local-only/tests/integration/myAccount/respondent-modifies-contact-info.cy.ts
index d0ecc4ec8c1..774d45f1218 100644
--- a/cypress/local-only/tests/integration/myAccount/respondent-modifies-contact-info.cy.ts
+++ b/cypress/local-only/tests/integration/myAccount/respondent-modifies-contact-info.cy.ts
@@ -7,7 +7,7 @@ import { respondentModifiesContactInfo } from '../../../../helpers/myAccount/res
 const BAR_NUMBER = 'WN7777';
 const USER = 'irspractitioner2';
 
-describe('a repondent modifies their address', () => {
+describe('a respondent modifies their address', () => {
   it('should generate a notice of change address for all cases associated with the respondent', function () {
     loginAsPetitioner();
     petitionerCreatesElectronicCase().then(docketNumber => {
diff --git a/web-client/src/views/AddPetitionerToCase/AddPetitionerToCase.tsx b/web-client/src/views/AddPetitionerToCase/AddPetitionerToCase.tsx
index bdd60a26472..8cfeef35310 100644
--- a/web-client/src/views/AddPetitionerToCase/AddPetitionerToCase.tsx
+++ b/web-client/src/views/AddPetitionerToCase/AddPetitionerToCase.tsx
@@ -51,7 +51,8 @@ export const AddPetitionerToCase = connect(
   }) {
     const type = 'contact';
     const bind = 'form';
-    const onBlur = 'validateAddPetitionerSequence';
+    const onBlur = () =>
+      validateAddPetitionerSequence({ preventAutoScroll: true });
 
     return (
       <>
diff --git a/web-client/src/views/EditPetitionerInformationInternal.tsx b/web-client/src/views/EditPetitionerInformationInternal.tsx
index 80e22d3673e..4743121abca 100644
--- a/web-client/src/views/EditPetitionerInformationInternal.tsx
+++ b/web-client/src/views/EditPetitionerInformationInternal.tsx
@@ -55,7 +55,8 @@ export const EditPetitionerInformationInternal = connect(
   }) {
     const type = 'contact';
     const bind = 'form';
-    const onBlur = 'validatePetitionerSequence';
+    const onBlur = () =>
+      validatePetitionerSequence({ preventAutoScroll: true });
 
     return (
       <>
diff --git a/web-client/src/views/Practitioners/PractitionerContactForm.tsx b/web-client/src/views/Practitioners/PractitionerContactForm.tsx
index f3eab1c22cf..1abb1a4c7b4 100644
--- a/web-client/src/views/Practitioners/PractitionerContactForm.tsx
+++ b/web-client/src/views/Practitioners/PractitionerContactForm.tsx
@@ -22,7 +22,6 @@ export const PractitionerContactForm = connect(
     changeCountryTypeSequence,
     COUNTRY_TYPES,
     form,
-    onBlurSequenceName,
     onBlurValidationSequence,
     onChangeSequenceName,
     onChangeUpdateSequence,
@@ -34,7 +33,7 @@ export const PractitionerContactForm = connect(
         <Country
           bind={bind}
           type={type}
-          onBlur={onBlurSequenceName}
+          onBlur={() => onBlurValidationSequence({ preventAutoScroll: true })}
           onChange={onChangeSequenceName}
           onChangeCountryType={changeCountryTypeSequence}
         />
@@ -42,14 +41,14 @@ export const PractitionerContactForm = connect(
           <Address
             bind={bind}
             type={type}
-            onBlur={onBlurSequenceName}
+            onBlur={() => onBlurValidationSequence({ preventAutoScroll: true })}
             onChange={onChangeSequenceName}
           />
         ) : (
           <InternationalAddress
             bind={bind}
             type={type}
-            onBlur={onBlurSequenceName}
+            onBlur={() => onBlurValidationSequence({ preventAutoScroll: true })}
             onChange={onChangeSequenceName}
           />
         )}
@@ -70,9 +69,9 @@ export const PractitionerContactForm = connect(
                 name="contact.phone"
                 type="text"
                 value={form.contact.phone || ''}
-                onBlur={() => {
-                  onBlurValidationSequence();
-                }}
+                onBlur={() =>
+                  onBlurValidationSequence({ preventAutoScroll: true })
+                }
                 onChange={e => {
                   onChangeUpdateSequence({
                     key: e.target.name,
diff --git a/web-client/src/views/UserContactEditForm.tsx b/web-client/src/views/UserContactEditForm.tsx
index 3caebda5438..b9254ab226f 100644
--- a/web-client/src/views/UserContactEditForm.tsx
+++ b/web-client/src/views/UserContactEditForm.tsx
@@ -30,7 +30,9 @@ export const UserContactEditForm = connect(
         <Country
           bind={bind}
           type={type}
-          onBlur={'validateUserContactSequence'}
+          onBlur={() =>
+            validateUserContactSequence({ preventAutoScroll: true })
+          }
           onChange={'updateFormValueSequence'}
           onChangeCountryType={'changeCountryTypeSequence'}
         />
@@ -38,14 +40,18 @@ export const UserContactEditForm = connect(
           <Address
             bind={bind}
             type={type}
-            onBlur={'validateUserContactSequence'}
+            onBlur={() =>
+              validateUserContactSequence({ preventAutoScroll: true })
+            }
             onChange={'updateFormValueSequence'}
           />
         ) : (
           <InternationalAddress
             bind={bind}
             type={type}
-            onBlur={'validateUserContactSequence'}
+            onBlur={() =>
+              validateUserContactSequence({ preventAutoScroll: true })
+            }
             onChange={'updateFormValueSequence'}
           />
         )}
@@ -64,9 +70,9 @@ export const UserContactEditForm = connect(
             name="contact.phone"
             type="text"
             value={form.contact.phone || ''}
-            onBlur={() => {
-              validateUserContactSequence();
-            }}
+            onBlur={() =>
+              validateUserContactSequence({ preventAutoScroll: true })
+            }
             onChange={e => {
               updateFormValueSequence({
                 key: e.target.name,

From b1699827b8b39be13442f8b6f6768a4b0ebd8614 Mon Sep 17 00:00:00 2001
From: Christopher Bisom <cbisom@flexion.us>
Date: Thu, 19 Sep 2024 09:48:57 -0400
Subject: [PATCH 06/33] 10074-bug: update PetitionPaymentForm

---
 .../src/views/CaseDetail/EditCaseDetails.tsx  |   4 +-
 .../views/CaseDetail/PetitionPaymentForm.tsx  | 305 +++++++++---------
 .../src/views/CaseDetailEdit/CaseInfo.tsx     |   4 +-
 .../StartCaseInternal/CaseInformation.tsx     |   6 +-
 4 files changed, 163 insertions(+), 156 deletions(-)

diff --git a/web-client/src/views/CaseDetail/EditCaseDetails.tsx b/web-client/src/views/CaseDetail/EditCaseDetails.tsx
index d3dc5af3adb..2dd28359692 100644
--- a/web-client/src/views/CaseDetail/EditCaseDetails.tsx
+++ b/web-client/src/views/CaseDetail/EditCaseDetails.tsx
@@ -87,9 +87,9 @@ export const EditCaseDetails = connect(
 
             <PetitionPaymentForm
               bind="form"
-              updateSequence={updateFormValueSequence}
-              validateSequence={validateCaseDetailsSequence}
               validationErrorsBind="validationErrors"
+              onUpdate={updateFormValueSequence}
+              onValidate={validateCaseDetailsSequence}
             />
           </div>
 
diff --git a/web-client/src/views/CaseDetail/PetitionPaymentForm.tsx b/web-client/src/views/CaseDetail/PetitionPaymentForm.tsx
index 47594cd5b8c..fd433b80d1b 100644
--- a/web-client/src/views/CaseDetail/PetitionPaymentForm.tsx
+++ b/web-client/src/views/CaseDetail/PetitionPaymentForm.tsx
@@ -6,172 +6,181 @@ import { sequences, state } from '@web-client/presenter/app.cerebral';
 import React from 'react';
 import classNames from 'classnames';
 
-export const PetitionPaymentForm = connect(
-  {
-    DATE_FORMATS: state.constants.DATE_FORMATS,
-    bind: state[props.bind],
-    formatAndUpdateDateFromDatePickerSequence:
-      sequences.formatAndUpdateDateFromDatePickerSequence,
-    paymentStatus: state.constants.PAYMENT_STATUS,
-    validationErrors: state[props.validationErrorsBind],
-  },
-  function PetitionPaymentForm({
-    bind,
-    DATE_FORMATS,
-    formatAndUpdateDateFromDatePickerSequence,
-    paymentStatus,
-    updateSequence,
-    validateSequence,
-    validationErrors,
-  }) {
-    return (
-      <>
-        <h4 className="margin-bottom-2">Petition Fee</h4>
-        <FormGroup errorText={validationErrors.petitionPaymentStatus}>
-          <fieldset className="usa-fieldset">
-            <legend className="usa-legend" id="fee-paid-legend">
-              Fee paid?
-            </legend>
-            <div className="usa-radio usa-radio__inline">
-              <input
-                aria-describedby="fee-paid-legend"
-                checked={bind.petitionPaymentStatus === paymentStatus.PAID}
-                className="usa-radio__input"
-                id="payment-status-paid"
-                name="petitionPaymentStatus"
-                type="radio"
-                value={paymentStatus.PAID}
-                onChange={e => {
-                  updateSequence({
-                    key: e.target.name,
-                    value: e.target.value,
-                  });
-                  validateSequence();
-                }}
-              />
-              <label
-                className="usa-radio__label"
-                data-testid="payment-status-paid-radio"
-                htmlFor="payment-status-paid"
-              >
-                {paymentStatus.PAID}
-              </label>
-            </div>
+interface ComponentProps {
+  onUpdate: (args: Record<string, any>) => void;
+  onValidate: (args: Record<string, any>) => void;
+}
 
-            <div className="usa-radio usa-radio__inline">
-              <input
-                aria-describedby="fee-paid-legend"
-                checked={bind.petitionPaymentStatus === paymentStatus.UNPAID}
-                className="usa-radio__input"
-                id="payment-status-unpaid"
-                name="petitionPaymentStatus"
-                type="radio"
-                value={paymentStatus.UNPAID}
-                onChange={e => {
-                  updateSequence({
-                    key: e.target.name,
-                    value: e.target.value,
-                  });
-                  validateSequence();
-                }}
-              />
-              <label
-                className="usa-radio__label"
-                data-testid="payment-status-unpaid-radio"
-                htmlFor="payment-status-unpaid"
-              >
-                {paymentStatus.UNPAID}
-              </label>
-            </div>
-
-            <div className="usa-radio usa-radio__inline">
-              <input
-                aria-describedby="fee-paid-legend"
-                checked={bind.petitionPaymentStatus === paymentStatus.WAIVED}
-                className="usa-radio__input"
-                id="payment-status-waived"
-                name="petitionPaymentStatus"
-                type="radio"
-                value={paymentStatus.WAIVED}
-                onChange={e => {
-                  updateSequence({
-                    key: e.target.name,
-                    value: e.target.value,
-                  });
-                  validateSequence();
-                }}
-              />
-              <label
-                className="usa-radio__label"
-                data-testid="payment-status-waived-radio"
-                htmlFor="payment-status-waived"
-              >
-                {paymentStatus.WAIVED}
-              </label>
-            </div>
-          </fieldset>
-        </FormGroup>
+// It would be better to use ComponentProps & CerebralProps, but
+// typing won't work nicely with CerebralProps. Hence the & any.
+const PetitionPaymentFormComponent: React.FC<ComponentProps & any> = ({
+  bind,
+  DATE_FORMATS,
+  formatAndUpdateDateFromDatePickerSequence,
+  onUpdate,
+  onValidate,
+  paymentStatus,
+  validationErrors,
+}) => {
+  return (
+    <>
+      <h4 className="margin-bottom-2">Petition Fee</h4>
+      <FormGroup errorText={validationErrors.petitionPaymentStatus}>
+        <fieldset className="usa-fieldset">
+          <legend className="usa-legend" id="fee-paid-legend">
+            Fee paid?
+          </legend>
+          <div className="usa-radio usa-radio__inline">
+            <input
+              aria-describedby="fee-paid-legend"
+              checked={bind.petitionPaymentStatus === paymentStatus.PAID}
+              className="usa-radio__input"
+              id="payment-status-paid"
+              name="petitionPaymentStatus"
+              type="radio"
+              value={paymentStatus.PAID}
+              onChange={e => {
+                onUpdate({
+                  key: e.target.name,
+                  value: e.target.value,
+                });
+                onValidate({ preventAutoScroll: true });
+              }}
+            />
+            <label
+              className="usa-radio__label"
+              data-testid="payment-status-paid-radio"
+              htmlFor="payment-status-paid"
+            >
+              {paymentStatus.PAID}
+            </label>
+          </div>
 
-        {bind.petitionPaymentStatus === paymentStatus.PAID && (
-          <>
-            <DateSelector
-              defaultValue={bind.petitionPaymentDate}
-              errorText={validationErrors.petitionPaymentDate}
-              id="payment-date"
-              label="Payment date"
+          <div className="usa-radio usa-radio__inline">
+            <input
+              aria-describedby="fee-paid-legend"
+              checked={bind.petitionPaymentStatus === paymentStatus.UNPAID}
+              className="usa-radio__input"
+              id="payment-status-unpaid"
+              name="petitionPaymentStatus"
+              type="radio"
+              value={paymentStatus.UNPAID}
               onChange={e => {
-                formatAndUpdateDateFromDatePickerSequence({
-                  key: 'petitionPaymentDate',
-                  toFormat: DATE_FORMATS.ISO,
+                onUpdate({
+                  key: e.target.name,
                   value: e.target.value,
                 });
-                validateSequence();
+                onValidate({ preventAutoScroll: true });
               }}
             />
+            <label
+              className="usa-radio__label"
+              data-testid="payment-status-unpaid-radio"
+              htmlFor="payment-status-unpaid"
+            >
+              {paymentStatus.UNPAID}
+            </label>
+          </div>
 
-            <FormGroup errorText={validationErrors.petitionPaymentMethod}>
-              <label className="usa-label" htmlFor="petition-payment-method">
-                Payment method
-              </label>
-              <input
-                className={classNames(
-                  'usa-input',
-                  validationErrors.petitionPaymentMethod && 'usa-input-error',
-                )}
-                id="petition-payment-method"
-                name="petitionPaymentMethod"
-                value={bind.petitionPaymentMethod || ''}
-                onBlur={() => validateSequence()}
-                onChange={e => {
-                  updateSequence({
-                    key: e.target.name,
-                    value: e.target.value,
-                  });
-                }}
-              />
-            </FormGroup>
-          </>
-        )}
+          <div className="usa-radio usa-radio__inline">
+            <input
+              aria-describedby="fee-paid-legend"
+              checked={bind.petitionPaymentStatus === paymentStatus.WAIVED}
+              className="usa-radio__input"
+              id="payment-status-waived"
+              name="petitionPaymentStatus"
+              type="radio"
+              value={paymentStatus.WAIVED}
+              onChange={e => {
+                onUpdate({
+                  key: e.target.name,
+                  value: e.target.value,
+                });
+                onValidate({ preventAutoScroll: true });
+              }}
+            />
+            <label
+              className="usa-radio__label"
+              data-testid="payment-status-waived-radio"
+              htmlFor="payment-status-waived"
+            >
+              {paymentStatus.WAIVED}
+            </label>
+          </div>
+        </fieldset>
+      </FormGroup>
 
-        {bind.petitionPaymentStatus === paymentStatus.WAIVED && (
+      {bind.petitionPaymentStatus === paymentStatus.PAID && (
+        <>
           <DateSelector
-            defaultValue={bind.petitionPaymentWaivedDate}
-            errorText={validationErrors.petitionPaymentWaivedDate}
-            id="payment-date-waived"
-            label="Date waived"
+            defaultValue={bind.petitionPaymentDate}
+            errorText={validationErrors.petitionPaymentDate}
+            id="payment-date"
+            label="Payment date"
             onChange={e => {
               formatAndUpdateDateFromDatePickerSequence({
-                key: 'petitionPaymentWaivedDate',
+                key: 'petitionPaymentDate',
                 toFormat: DATE_FORMATS.ISO,
                 value: e.target.value,
               });
-              validateSequence();
+              onValidate({ preventAutoScroll: true });
             }}
           />
-        )}
-      </>
-    );
+
+          <FormGroup errorText={validationErrors.petitionPaymentMethod}>
+            <label className="usa-label" htmlFor="petition-payment-method">
+              Payment method
+            </label>
+            <input
+              className={classNames(
+                'usa-input',
+                validationErrors.petitionPaymentMethod && 'usa-input-error',
+              )}
+              id="petition-payment-method"
+              name="petitionPaymentMethod"
+              value={bind.petitionPaymentMethod || ''}
+              onBlur={() => onValidate({ preventAutoScroll: true })}
+              onChange={e => {
+                onUpdate({
+                  key: e.target.name,
+                  value: e.target.value,
+                });
+              }}
+            />
+          </FormGroup>
+        </>
+      )}
+
+      {bind.petitionPaymentStatus === paymentStatus.WAIVED && (
+        <DateSelector
+          defaultValue={bind.petitionPaymentWaivedDate}
+          errorText={validationErrors.petitionPaymentWaivedDate}
+          id="payment-date-waived"
+          label="Date waived"
+          onChange={e => {
+            formatAndUpdateDateFromDatePickerSequence({
+              key: 'petitionPaymentWaivedDate',
+              toFormat: DATE_FORMATS.ISO,
+              value: e.target.value,
+            });
+            onValidate({ preventAutoScroll: true });
+          }}
+        />
+      )}
+    </>
+  );
+};
+
+export const PetitionPaymentForm = connect(
+  {
+    DATE_FORMATS: state.constants.DATE_FORMATS,
+    bind: state[props.bind],
+    formatAndUpdateDateFromDatePickerSequence:
+      sequences.formatAndUpdateDateFromDatePickerSequence,
+    paymentStatus: state.constants.PAYMENT_STATUS,
+    validationErrors: state[props.validationErrorsBind],
   },
+  PetitionPaymentFormComponent,
 );
 
 PetitionPaymentForm.displayName = 'PetitionPaymentForm';
diff --git a/web-client/src/views/CaseDetailEdit/CaseInfo.tsx b/web-client/src/views/CaseDetailEdit/CaseInfo.tsx
index 8151b68d79a..05a48145fdb 100644
--- a/web-client/src/views/CaseDetailEdit/CaseInfo.tsx
+++ b/web-client/src/views/CaseDetailEdit/CaseInfo.tsx
@@ -222,9 +222,9 @@ export const CaseInfo = connect(
 
         <PetitionPaymentForm
           bind="form"
-          updateSequence={updatePetitionPaymentFormValueSequence}
-          validateSequence={validateCaseDetailSequence}
           validationErrorsBind="validationErrors"
+          onUpdate={updatePetitionPaymentFormValueSequence}
+          onValidate={validateCaseDetailSequence}
         />
 
         {caseDetailEditHelper.showOrderForFilingFee && (
diff --git a/web-client/src/views/StartCaseInternal/CaseInformation.tsx b/web-client/src/views/StartCaseInternal/CaseInformation.tsx
index e7f25e27851..9cd583cd763 100644
--- a/web-client/src/views/StartCaseInternal/CaseInformation.tsx
+++ b/web-client/src/views/StartCaseInternal/CaseInformation.tsx
@@ -181,11 +181,9 @@ export const CaseInformation = connect(
         </FormGroup>
         <PetitionPaymentForm
           bind="form"
-          updateSequence={updatePetitionPaymentFormValueSequence}
-          validateSequence={() =>
-            validatePetitionFromPaperSequence({ preventAutoScroll: true })
-          }
           validationErrorsBind="validationErrors"
+          onUpdate={updatePetitionPaymentFormValueSequence}
+          onValidate={validatePetitionFromPaperSequence}
         />
         {startCaseInternalHelper.showOrderForFilingFee && (
           <div className="order-checkbox">

From bb241d67c8ddb0ec2d5ec184502bfb6f04af5120 Mon Sep 17 00:00:00 2001
From: Christopher Bisom <cbisom@flexion.us>
Date: Thu, 19 Sep 2024 10:01:42 -0400
Subject: [PATCH 07/33] 10074-bug: add allowEmptyString so that
 petitionPaymentMethod is actually validated rather than causing a 400 error

---
 web-client/src/views/CaseDetail/PetitionPaymentForm.tsx | 1 +
 1 file changed, 1 insertion(+)

diff --git a/web-client/src/views/CaseDetail/PetitionPaymentForm.tsx b/web-client/src/views/CaseDetail/PetitionPaymentForm.tsx
index fd433b80d1b..00c38791217 100644
--- a/web-client/src/views/CaseDetail/PetitionPaymentForm.tsx
+++ b/web-client/src/views/CaseDetail/PetitionPaymentForm.tsx
@@ -142,6 +142,7 @@ const PetitionPaymentFormComponent: React.FC<ComponentProps & any> = ({
               onBlur={() => onValidate({ preventAutoScroll: true })}
               onChange={e => {
                 onUpdate({
+                  allowEmptyString: true,
                   key: e.target.name,
                   value: e.target.value,
                 });

From c4cb6e8cbc138cf6445a6366ec32f09f39ae7b04 Mon Sep 17 00:00:00 2001
From: Christopher Bisom <cbisom@flexion.us>
Date: Thu, 19 Sep 2024 10:13:55 -0400
Subject: [PATCH 08/33] 10074-bug: rename onValidate to validateFormData

---
 .../src/views/CaseDetail/EditCaseDetails.tsx     |  4 ++--
 .../src/views/CaseDetail/PetitionPaymentForm.tsx | 16 ++++++++--------
 .../src/views/CaseDetailEdit/CaseDetailEdit.tsx  |  2 +-
 web-client/src/views/CaseDetailEdit/CaseInfo.tsx |  2 +-
 web-client/src/views/IRSNotice.tsx               |  8 ++++----
 .../src/views/StartCase/CaseTypeSelect.tsx       |  8 ++++----
 .../src/views/StartCase/StartCaseStep2.tsx       |  4 ++--
 .../views/StartCaseInternal/CaseInformation.tsx  |  2 +-
 .../StartCaseInternal/StartCaseInternal.tsx      | 12 ++----------
 9 files changed, 25 insertions(+), 33 deletions(-)

diff --git a/web-client/src/views/CaseDetail/EditCaseDetails.tsx b/web-client/src/views/CaseDetail/EditCaseDetails.tsx
index 2dd28359692..b85200003dc 100644
--- a/web-client/src/views/CaseDetail/EditCaseDetails.tsx
+++ b/web-client/src/views/CaseDetail/EditCaseDetails.tsx
@@ -45,7 +45,7 @@ export const EditCaseDetails = connect(
               <h4 className="margin-bottom-2">IRS Notice/Case</h4>
               <IRSNotice
                 shouldStartWithBlankStatistic={false}
-                validationFunction={validateCaseDetailsSequence}
+                validateFormData={validateCaseDetailsSequence}
               />
 
               <ProcedureType
@@ -87,9 +87,9 @@ export const EditCaseDetails = connect(
 
             <PetitionPaymentForm
               bind="form"
+              validateFormData={validateCaseDetailsSequence}
               validationErrorsBind="validationErrors"
               onUpdate={updateFormValueSequence}
-              onValidate={validateCaseDetailsSequence}
             />
           </div>
 
diff --git a/web-client/src/views/CaseDetail/PetitionPaymentForm.tsx b/web-client/src/views/CaseDetail/PetitionPaymentForm.tsx
index 00c38791217..d5861d9b518 100644
--- a/web-client/src/views/CaseDetail/PetitionPaymentForm.tsx
+++ b/web-client/src/views/CaseDetail/PetitionPaymentForm.tsx
@@ -8,7 +8,7 @@ import classNames from 'classnames';
 
 interface ComponentProps {
   onUpdate: (args: Record<string, any>) => void;
-  onValidate: (args: Record<string, any>) => void;
+  validateFormData: (args: Record<string, any>) => void;
 }
 
 // It would be better to use ComponentProps & CerebralProps, but
@@ -18,8 +18,8 @@ const PetitionPaymentFormComponent: React.FC<ComponentProps & any> = ({
   DATE_FORMATS,
   formatAndUpdateDateFromDatePickerSequence,
   onUpdate,
-  onValidate,
   paymentStatus,
+  validateFormData,
   validationErrors,
 }) => {
   return (
@@ -44,7 +44,7 @@ const PetitionPaymentFormComponent: React.FC<ComponentProps & any> = ({
                   key: e.target.name,
                   value: e.target.value,
                 });
-                onValidate({ preventAutoScroll: true });
+                validateFormData({ preventAutoScroll: true });
               }}
             />
             <label
@@ -70,7 +70,7 @@ const PetitionPaymentFormComponent: React.FC<ComponentProps & any> = ({
                   key: e.target.name,
                   value: e.target.value,
                 });
-                onValidate({ preventAutoScroll: true });
+                validateFormData({ preventAutoScroll: true });
               }}
             />
             <label
@@ -96,7 +96,7 @@ const PetitionPaymentFormComponent: React.FC<ComponentProps & any> = ({
                   key: e.target.name,
                   value: e.target.value,
                 });
-                onValidate({ preventAutoScroll: true });
+                validateFormData({ preventAutoScroll: true });
               }}
             />
             <label
@@ -123,7 +123,7 @@ const PetitionPaymentFormComponent: React.FC<ComponentProps & any> = ({
                 toFormat: DATE_FORMATS.ISO,
                 value: e.target.value,
               });
-              onValidate({ preventAutoScroll: true });
+              validateFormData({ preventAutoScroll: true });
             }}
           />
 
@@ -139,7 +139,7 @@ const PetitionPaymentFormComponent: React.FC<ComponentProps & any> = ({
               id="petition-payment-method"
               name="petitionPaymentMethod"
               value={bind.petitionPaymentMethod || ''}
-              onBlur={() => onValidate({ preventAutoScroll: true })}
+              onBlur={() => validateFormData({ preventAutoScroll: true })}
               onChange={e => {
                 onUpdate({
                   allowEmptyString: true,
@@ -164,7 +164,7 @@ const PetitionPaymentFormComponent: React.FC<ComponentProps & any> = ({
               toFormat: DATE_FORMATS.ISO,
               value: e.target.value,
             });
-            onValidate({ preventAutoScroll: true });
+            validateFormData({ preventAutoScroll: true });
           }}
         />
       )}
diff --git a/web-client/src/views/CaseDetailEdit/CaseDetailEdit.tsx b/web-client/src/views/CaseDetailEdit/CaseDetailEdit.tsx
index cbc76ab7e98..e647306b17e 100644
--- a/web-client/src/views/CaseDetailEdit/CaseDetailEdit.tsx
+++ b/web-client/src/views/CaseDetailEdit/CaseDetailEdit.tsx
@@ -48,7 +48,7 @@ export const CaseDetailEdit = connect(
             title="IRS Notice"
           >
             <div className="blue-container">
-              <IRSNotice validationFunction={validateCaseDetailSequence} />
+              <IRSNotice validateFormData={validateCaseDetailSequence} />
             </div>
           </Tab>
         </Tabs>
diff --git a/web-client/src/views/CaseDetailEdit/CaseInfo.tsx b/web-client/src/views/CaseDetailEdit/CaseInfo.tsx
index 05a48145fdb..155f07e7efe 100644
--- a/web-client/src/views/CaseDetailEdit/CaseInfo.tsx
+++ b/web-client/src/views/CaseDetailEdit/CaseInfo.tsx
@@ -222,9 +222,9 @@ export const CaseInfo = connect(
 
         <PetitionPaymentForm
           bind="form"
+          validateFormData={validateCaseDetailSequence}
           validationErrorsBind="validationErrors"
           onUpdate={updatePetitionPaymentFormValueSequence}
-          onValidate={validateCaseDetailSequence}
         />
 
         {caseDetailEditHelper.showOrderForFilingFee && (
diff --git a/web-client/src/views/IRSNotice.tsx b/web-client/src/views/IRSNotice.tsx
index cf073285d75..11f80e4ec11 100644
--- a/web-client/src/views/IRSNotice.tsx
+++ b/web-client/src/views/IRSNotice.tsx
@@ -21,8 +21,8 @@ export const IRSNotice = connect(
     showModal: state.modal.showModal,
     statisticsFormHelper: state.statisticsFormHelper,
     updateFormValueSequence: sequences.updateFormValueSequence,
+    validateFormData: props.validateFormData,
     validationErrors: state.validationErrors,
-    validationFunction: props.validationFunction,
   },
   function IRSNotice({
     caseDetailEditHelper,
@@ -35,8 +35,8 @@ export const IRSNotice = connect(
     showModal,
     statisticsFormHelper,
     updateFormValueSequence,
+    validateFormData,
     validationErrors,
-    validationFunction,
   }) {
     const renderIrsNoticeRadios = () => {
       return (
@@ -119,7 +119,7 @@ export const IRSNotice = connect(
               toFormat: constants.DATE_FORMATS.ISO,
               value: e.target.value,
             });
-            validationFunction({ preventAutoScroll: true });
+            validateFormData({ preventAutoScroll: true });
           }}
         />
       );
@@ -133,7 +133,7 @@ export const IRSNotice = connect(
           allowDefaultOption={true}
           caseTypes={constants.CASE_TYPES}
           legend="Type of case"
-          validationFunction={validationFunction}
+          validateFormData={validateFormData}
           value={form.caseType}
           onChange="updateFormValueSequence"
           onChangePreValidation="refreshStatisticsSequence"
diff --git a/web-client/src/views/StartCase/CaseTypeSelect.tsx b/web-client/src/views/StartCase/CaseTypeSelect.tsx
index 560b62cae9a..1c48620fbd9 100644
--- a/web-client/src/views/StartCase/CaseTypeSelect.tsx
+++ b/web-client/src/views/StartCase/CaseTypeSelect.tsx
@@ -19,9 +19,9 @@ export const CaseTypeSelect = connect(
     onChangePreValidation: sequences[props.onChangePreValidation],
     rawOnChange: props.onChange,
     refProp: props.refProp,
+    validateFormData: props.validateFormData,
     validationError: props.validationError,
     validationErrors: state.validationErrors,
-    validationFunction: props.validationFunction,
     value: props.value,
   },
   function CaseTypeSelect({
@@ -37,9 +37,9 @@ export const CaseTypeSelect = connect(
     onChangePreValidation,
     rawOnChange,
     refProp,
+    validateFormData,
     validationError,
     validationErrors,
-    validationFunction,
     value,
   }) {
     return (
@@ -69,8 +69,8 @@ export const CaseTypeSelect = connect(
                   value: e.target.value,
                 });
                 if (onChangePreValidation) onChangePreValidation();
-                if (validationFunction)
-                  validationFunction({ preventAutoScroll: true });
+                if (validateFormData)
+                  validateFormData({ preventAutoScroll: true });
               }}
             >
               {allowDefaultOption && <option value="">-- Select --</option>}
diff --git a/web-client/src/views/StartCase/StartCaseStep2.tsx b/web-client/src/views/StartCase/StartCaseStep2.tsx
index 72026589532..09bd9d1f8d6 100644
--- a/web-client/src/views/StartCase/StartCaseStep2.tsx
+++ b/web-client/src/views/StartCase/StartCaseStep2.tsx
@@ -145,7 +145,7 @@ export const StartCaseStep2 = connect(
                   allowDefaultOption={true}
                   caseTypes={caseTypeDescriptionHelper.caseTypes}
                   legend="Type of notice / case"
-                  validationFunction={validateStartCaseWizardSequence}
+                  validateFormData={validateStartCaseWizardSequence}
                   value={form.caseType}
                   onChange="updateFormValueSequence"
                 />
@@ -193,7 +193,7 @@ export const StartCaseStep2 = connect(
                 className="margin-bottom-0"
                 legend="Which topic most closely matches your complaint with the
                 IRS?"
-                validationFunction={validateStartCaseWizardSequence}
+                validateFormData={validateStartCaseWizardSequence}
                 value={form.caseType}
                 onChange="updateFormValueSequence"
               />
diff --git a/web-client/src/views/StartCaseInternal/CaseInformation.tsx b/web-client/src/views/StartCaseInternal/CaseInformation.tsx
index 9cd583cd763..98b3e7bfa55 100644
--- a/web-client/src/views/StartCaseInternal/CaseInformation.tsx
+++ b/web-client/src/views/StartCaseInternal/CaseInformation.tsx
@@ -181,9 +181,9 @@ export const CaseInformation = connect(
         </FormGroup>
         <PetitionPaymentForm
           bind="form"
+          validateFormData={validatePetitionFromPaperSequence}
           validationErrorsBind="validationErrors"
           onUpdate={updatePetitionPaymentFormValueSequence}
-          onValidate={validatePetitionFromPaperSequence}
         />
         {startCaseInternalHelper.showOrderForFilingFee && (
           <div className="order-checkbox">
diff --git a/web-client/src/views/StartCaseInternal/StartCaseInternal.tsx b/web-client/src/views/StartCaseInternal/StartCaseInternal.tsx
index 812a57fb005..37575d0f803 100644
--- a/web-client/src/views/StartCaseInternal/StartCaseInternal.tsx
+++ b/web-client/src/views/StartCaseInternal/StartCaseInternal.tsx
@@ -78,11 +78,7 @@ export const StartCaseInternal = connect(
                   >
                     <div className="blue-container">
                       <IRSNotice
-                        validationFunction={() =>
-                          validateCaseDetailSequence({
-                            preventAutoScroll: true,
-                          })
-                        }
+                        validateFormData={validateCaseDetailSequence}
                       />
                     </div>
                   </Tab>
@@ -93,11 +89,7 @@ export const StartCaseInternal = connect(
                   documentTabs={startCaseHelper.documentTabs}
                   documentType={documentSelectedForScan}
                   title="Add Document(s)"
-                  validateSequence={() =>
-                    validatePetitionFromPaperSequence({
-                      preventAutoScroll: true,
-                    })
-                  }
+                  validateSequence={validatePetitionFromPaperSequence}
                 />
               </div>
             </div>

From cc5e87b8f06d017bbdd87d35699c1328c0849bf5 Mon Sep 17 00:00:00 2001
From: Christopher Bisom <cbisom@flexion.us>
Date: Thu, 19 Sep 2024 10:45:08 -0400
Subject: [PATCH 09/33] 10074-bug: fix integration test

---
 .../integration-tests/journey/petitionsClerkUpdatesCaseDetail.ts | 1 +
 1 file changed, 1 insertion(+)

diff --git a/web-client/integration-tests/journey/petitionsClerkUpdatesCaseDetail.ts b/web-client/integration-tests/journey/petitionsClerkUpdatesCaseDetail.ts
index efb273073ce..2b630a35a9d 100644
--- a/web-client/integration-tests/journey/petitionsClerkUpdatesCaseDetail.ts
+++ b/web-client/integration-tests/journey/petitionsClerkUpdatesCaseDetail.ts
@@ -143,6 +143,7 @@ export const petitionsClerkUpdatesCaseDetail = cerebralTest => {
     });
     expect(cerebralTest.getState('alertError')).toEqual({
       messages: ['Select a case type', 'Select a case procedure'],
+      preventAutoScroll: false,
       title: 'Please correct the following errors on the page:',
     });
 

From 4bf80c7aab1204122d0f932e22a833aa63851f24 Mon Sep 17 00:00:00 2001
From: Christopher Bisom <cbisom@flexion.us>
Date: Thu, 19 Sep 2024 16:51:18 -0400
Subject: [PATCH 10/33] 10074-bug: update trial sessions form to keep
 errornotification in sync with error validations, and rename
 preventAutoScroll to isSubmitting in most places since that is more
 meaningful/relevant

---
 .../journey/petitionsClerkUpdatesCaseDetail.ts |  2 +-
 .../actions/setValidationAlertErrorsAction.ts  |  2 +-
 .../submitPetitionFromPaperSequence.ts         |  7 +------
 .../validatePetitionFromPaperSequence.ts       |  7 +------
 .../sequences/validateTrialSessionSequence.ts  |  3 ++-
 .../AddPetitionerToCase.tsx                    |  3 +--
 .../views/CaseDetail/PetitionPaymentForm.tsx   | 12 ++++++------
 .../views/CaseDetailEdit/PartyInformation.tsx  |  4 +---
 .../EditPetitionerInformationInternal.tsx      |  3 +--
 web-client/src/views/IRSNotice.tsx             |  2 +-
 .../Practitioners/PractitionerContactForm.tsx  | 10 ++++------
 .../src/views/StartCase/CaseTypeSelect.tsx     |  3 +--
 .../CalculatePenaltiesModal.tsx                |  2 +-
 .../StartCaseInternal/CaseInformation.tsx      | 12 ++++++------
 .../src/views/StartCaseInternal/Parties.tsx    |  4 ++--
 .../views/StartCaseInternal/StatisticsForm.tsx |  4 ++--
 .../TrialSessions/SessionInformationForm.tsx   | 18 +++++++++++-------
 web-client/src/views/UserContactEditForm.tsx   | 16 ++++------------
 18 files changed, 47 insertions(+), 67 deletions(-)

diff --git a/web-client/integration-tests/journey/petitionsClerkUpdatesCaseDetail.ts b/web-client/integration-tests/journey/petitionsClerkUpdatesCaseDetail.ts
index 2b630a35a9d..566504508da 100644
--- a/web-client/integration-tests/journey/petitionsClerkUpdatesCaseDetail.ts
+++ b/web-client/integration-tests/journey/petitionsClerkUpdatesCaseDetail.ts
@@ -142,8 +142,8 @@ export const petitionsClerkUpdatesCaseDetail = cerebralTest => {
       procedureType: 'Select a case procedure',
     });
     expect(cerebralTest.getState('alertError')).toEqual({
+      isSubmitting: false,
       messages: ['Select a case type', 'Select a case procedure'],
-      preventAutoScroll: false,
       title: 'Please correct the following errors on the page:',
     });
 
diff --git a/web-client/src/presenter/actions/setValidationAlertErrorsAction.ts b/web-client/src/presenter/actions/setValidationAlertErrorsAction.ts
index 3dd3263d756..8a75f62ef44 100644
--- a/web-client/src/presenter/actions/setValidationAlertErrorsAction.ts
+++ b/web-client/src/presenter/actions/setValidationAlertErrorsAction.ts
@@ -86,7 +86,7 @@ export const setValidationAlertErrorsAction = ({
           }
         }),
     ),
-    preventAutoScroll: props.preventAutoScroll || false,
+    preventAutoScroll: props.isSubmitting || false,
     title: 'Please correct the following errors on the page:',
   };
   store.set(state.alertError, alertError);
diff --git a/web-client/src/presenter/sequences/submitPetitionFromPaperSequence.ts b/web-client/src/presenter/sequences/submitPetitionFromPaperSequence.ts
index 8aff399b9e9..ae19336842c 100644
--- a/web-client/src/presenter/sequences/submitPetitionFromPaperSequence.ts
+++ b/web-client/src/presenter/sequences/submitPetitionFromPaperSequence.ts
@@ -8,7 +8,6 @@ import { filterEmptyStatisticsAction } from '../actions/StartCaseInternal/filter
 import { getPetitionIdAction } from '../actions/getPetitionIdAction';
 import { navigateToReviewSavedPetitionAction } from '../actions/CaseDetailEdit/navigateToReviewSavedPetitionAction';
 import { openFileUploadErrorModal } from '../actions/openFileUploadErrorModal';
-import { setAlertErrorAction } from '../actions/setAlertErrorAction';
 import { setCaseAction } from '../actions/setCaseAction';
 import { setCaseTypeAction } from '../actions/setCaseTypeAction';
 import { setDocketEntryIdAction } from '../actions/setDocketEntryIdAction';
@@ -33,11 +32,7 @@ export const submitPetitionFromPaperSequence = [
       filterEmptyStatisticsAction,
       validatePetitionFromPaperAction,
       {
-        error: [
-          setAlertErrorAction,
-          setValidationErrorsAction,
-          setValidationAlertErrorsAction,
-        ],
+        error: [setValidationErrorsAction, setValidationAlertErrorsAction],
         success: [
           setupCurrentPageAction('Interstitial'),
           stopShowValidationAction,
diff --git a/web-client/src/presenter/sequences/validatePetitionFromPaperSequence.ts b/web-client/src/presenter/sequences/validatePetitionFromPaperSequence.ts
index 465e67ab30d..77f59f602a5 100644
--- a/web-client/src/presenter/sequences/validatePetitionFromPaperSequence.ts
+++ b/web-client/src/presenter/sequences/validatePetitionFromPaperSequence.ts
@@ -1,6 +1,5 @@
 import { clearAlertsAction } from '../actions/clearAlertsAction';
 import { clearErrorAlertsAction } from '../actions/clearErrorAlertsAction';
-import { setAlertErrorAction } from '../actions/setAlertErrorAction';
 import { setValidationAlertErrorsAction } from '../actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
 import { shouldValidateAction } from '../actions/shouldValidateAction';
@@ -13,11 +12,7 @@ export const validatePetitionFromPaperSequence = [
     validate: [
       validatePetitionFromPaperAction,
       {
-        error: [
-          setAlertErrorAction,
-          setValidationErrorsAction,
-          setValidationAlertErrorsAction,
-        ],
+        error: [setValidationErrorsAction, setValidationAlertErrorsAction],
         success: [clearAlertsAction, clearErrorAlertsAction],
       },
     ],
diff --git a/web-client/src/presenter/sequences/validateTrialSessionSequence.ts b/web-client/src/presenter/sequences/validateTrialSessionSequence.ts
index 5a2213260c6..5af4ab4f4ac 100644
--- a/web-client/src/presenter/sequences/validateTrialSessionSequence.ts
+++ b/web-client/src/presenter/sequences/validateTrialSessionSequence.ts
@@ -1,6 +1,7 @@
 import { clearAlertsAction } from '../actions/clearAlertsAction';
 import { computeTrialSessionFormDataAction } from '../actions/TrialSession/computeTrialSessionFormDataAction';
 import { getJudgesChambersSequence } from '@web-client/presenter/sequences/getJudgesChambersSequence';
+import { setValidationAlertErrorsAction } from '@web-client/presenter/actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
 import { shouldValidateAction } from '../actions/shouldValidateAction';
 import { validateTrialSessionAction } from '../actions/TrialSession/validateTrialSessionAction';
@@ -14,7 +15,7 @@ export const validateTrialSessionSequence = [
       computeTrialSessionFormDataAction,
       validateTrialSessionAction,
       {
-        error: [setValidationErrorsAction],
+        error: [setValidationErrorsAction, setValidationAlertErrorsAction],
         success: [clearAlertsAction],
       },
     ],
diff --git a/web-client/src/views/AddPetitionerToCase/AddPetitionerToCase.tsx b/web-client/src/views/AddPetitionerToCase/AddPetitionerToCase.tsx
index 8cfeef35310..5d5cda96b62 100644
--- a/web-client/src/views/AddPetitionerToCase/AddPetitionerToCase.tsx
+++ b/web-client/src/views/AddPetitionerToCase/AddPetitionerToCase.tsx
@@ -51,8 +51,7 @@ export const AddPetitionerToCase = connect(
   }) {
     const type = 'contact';
     const bind = 'form';
-    const onBlur = () =>
-      validateAddPetitionerSequence({ preventAutoScroll: true });
+    const onBlur = () => validateAddPetitionerSequence({ isSubmitting: true });
 
     return (
       <>
diff --git a/web-client/src/views/CaseDetail/PetitionPaymentForm.tsx b/web-client/src/views/CaseDetail/PetitionPaymentForm.tsx
index d5861d9b518..36a63ed2808 100644
--- a/web-client/src/views/CaseDetail/PetitionPaymentForm.tsx
+++ b/web-client/src/views/CaseDetail/PetitionPaymentForm.tsx
@@ -44,7 +44,7 @@ const PetitionPaymentFormComponent: React.FC<ComponentProps & any> = ({
                   key: e.target.name,
                   value: e.target.value,
                 });
-                validateFormData({ preventAutoScroll: true });
+                validateFormData({ isSubmitting: true });
               }}
             />
             <label
@@ -70,7 +70,7 @@ const PetitionPaymentFormComponent: React.FC<ComponentProps & any> = ({
                   key: e.target.name,
                   value: e.target.value,
                 });
-                validateFormData({ preventAutoScroll: true });
+                validateFormData({ isSubmitting: true });
               }}
             />
             <label
@@ -96,7 +96,7 @@ const PetitionPaymentFormComponent: React.FC<ComponentProps & any> = ({
                   key: e.target.name,
                   value: e.target.value,
                 });
-                validateFormData({ preventAutoScroll: true });
+                validateFormData({ isSubmitting: true });
               }}
             />
             <label
@@ -123,7 +123,7 @@ const PetitionPaymentFormComponent: React.FC<ComponentProps & any> = ({
                 toFormat: DATE_FORMATS.ISO,
                 value: e.target.value,
               });
-              validateFormData({ preventAutoScroll: true });
+              validateFormData({ isSubmitting: true });
             }}
           />
 
@@ -139,7 +139,7 @@ const PetitionPaymentFormComponent: React.FC<ComponentProps & any> = ({
               id="petition-payment-method"
               name="petitionPaymentMethod"
               value={bind.petitionPaymentMethod || ''}
-              onBlur={() => validateFormData({ preventAutoScroll: true })}
+              onBlur={() => validateFormData({ isSubmitting: true })}
               onChange={e => {
                 onUpdate({
                   allowEmptyString: true,
@@ -164,7 +164,7 @@ const PetitionPaymentFormComponent: React.FC<ComponentProps & any> = ({
               toFormat: DATE_FORMATS.ISO,
               value: e.target.value,
             });
-            validateFormData({ preventAutoScroll: true });
+            validateFormData({ isSubmitting: true });
           }}
         />
       )}
diff --git a/web-client/src/views/CaseDetailEdit/PartyInformation.tsx b/web-client/src/views/CaseDetailEdit/PartyInformation.tsx
index a52690ce86a..14528640f47 100644
--- a/web-client/src/views/CaseDetailEdit/PartyInformation.tsx
+++ b/web-client/src/views/CaseDetailEdit/PartyInformation.tsx
@@ -89,9 +89,7 @@ export const PartyInformation = connect(
               showPrimaryContact={caseDetailEditHelper.showPrimaryContact}
               showSecondaryContact={caseDetailEditHelper.showSecondaryContact}
               useSameAsPrimary={true}
-              onBlur={() =>
-                validateCaseDetailSequence({ preventAutoScroll: true })
-              }
+              onBlur={() => validateCaseDetailSequence({ isSubmitting: true })}
               onChange="updateFormValueAndCaseCaptionSequence"
             />
           </div>
diff --git a/web-client/src/views/EditPetitionerInformationInternal.tsx b/web-client/src/views/EditPetitionerInformationInternal.tsx
index 4743121abca..9829e5e2364 100644
--- a/web-client/src/views/EditPetitionerInformationInternal.tsx
+++ b/web-client/src/views/EditPetitionerInformationInternal.tsx
@@ -55,8 +55,7 @@ export const EditPetitionerInformationInternal = connect(
   }) {
     const type = 'contact';
     const bind = 'form';
-    const onBlur = () =>
-      validatePetitionerSequence({ preventAutoScroll: true });
+    const onBlur = () => validatePetitionerSequence({ isSubmitting: true });
 
     return (
       <>
diff --git a/web-client/src/views/IRSNotice.tsx b/web-client/src/views/IRSNotice.tsx
index c3338d975f3..37059f2c97d 100644
--- a/web-client/src/views/IRSNotice.tsx
+++ b/web-client/src/views/IRSNotice.tsx
@@ -123,7 +123,7 @@ export const IRSNotice = connect<IrsNoticeType, typeof irsNoticeDependencies>(
               toFormat: constants.DATE_FORMATS.ISO,
               value: e.target.value,
             });
-            validateFormData({ preventAutoScroll: true });
+            validateFormData({ isSubmitting: true });
           }}
         />
       );
diff --git a/web-client/src/views/Practitioners/PractitionerContactForm.tsx b/web-client/src/views/Practitioners/PractitionerContactForm.tsx
index 1abb1a4c7b4..4282a54d3e2 100644
--- a/web-client/src/views/Practitioners/PractitionerContactForm.tsx
+++ b/web-client/src/views/Practitioners/PractitionerContactForm.tsx
@@ -33,7 +33,7 @@ export const PractitionerContactForm = connect(
         <Country
           bind={bind}
           type={type}
-          onBlur={() => onBlurValidationSequence({ preventAutoScroll: true })}
+          onBlur={() => onBlurValidationSequence({ isSubmitting: true })}
           onChange={onChangeSequenceName}
           onChangeCountryType={changeCountryTypeSequence}
         />
@@ -41,14 +41,14 @@ export const PractitionerContactForm = connect(
           <Address
             bind={bind}
             type={type}
-            onBlur={() => onBlurValidationSequence({ preventAutoScroll: true })}
+            onBlur={() => onBlurValidationSequence({ isSubmitting: true })}
             onChange={onChangeSequenceName}
           />
         ) : (
           <InternationalAddress
             bind={bind}
             type={type}
-            onBlur={() => onBlurValidationSequence({ preventAutoScroll: true })}
+            onBlur={() => onBlurValidationSequence({ isSubmitting: true })}
             onChange={onChangeSequenceName}
           />
         )}
@@ -69,9 +69,7 @@ export const PractitionerContactForm = connect(
                 name="contact.phone"
                 type="text"
                 value={form.contact.phone || ''}
-                onBlur={() =>
-                  onBlurValidationSequence({ preventAutoScroll: true })
-                }
+                onBlur={() => onBlurValidationSequence({ isSubmitting: true })}
                 onChange={e => {
                   onChangeUpdateSequence({
                     key: e.target.name,
diff --git a/web-client/src/views/StartCase/CaseTypeSelect.tsx b/web-client/src/views/StartCase/CaseTypeSelect.tsx
index 694644888e0..59c7af1b446 100644
--- a/web-client/src/views/StartCase/CaseTypeSelect.tsx
+++ b/web-client/src/views/StartCase/CaseTypeSelect.tsx
@@ -79,8 +79,7 @@ export const CaseTypeSelect = connect<
                   value: e.target.value,
                 });
                 if (onChangePreValidation) onChangePreValidation();
-                if (validateFormData)
-                  validateFormData({ preventAutoScroll: true });
+                if (validateFormData) validateFormData({ isSubmitting: true });
               }}
             >
               {allowDefaultOption && <option value="">-- Select --</option>}
diff --git a/web-client/src/views/StartCaseInternal/CalculatePenaltiesModal.tsx b/web-client/src/views/StartCaseInternal/CalculatePenaltiesModal.tsx
index ba43b63e5e9..2e41d11c153 100644
--- a/web-client/src/views/StartCaseInternal/CalculatePenaltiesModal.tsx
+++ b/web-client/src/views/StartCaseInternal/CalculatePenaltiesModal.tsx
@@ -40,7 +40,7 @@ export const CalculatePenaltiesModal = connect(
         cancelLabel="Cancel"
         cancelSequence={cancelSequence}
         confirmLabel="Calculate and Save"
-        confirmSequence={confirmSequence}
+        confirmSequence={() => confirmSequence({ isSubmitting: true })}
         title={title}
       >
         {errors &&
diff --git a/web-client/src/views/StartCaseInternal/CaseInformation.tsx b/web-client/src/views/StartCaseInternal/CaseInformation.tsx
index 98b3e7bfa55..690791c531b 100644
--- a/web-client/src/views/StartCaseInternal/CaseInformation.tsx
+++ b/web-client/src/views/StartCaseInternal/CaseInformation.tsx
@@ -53,7 +53,7 @@ export const CaseInformation = connect(
               toFormat: DATE_FORMATS.ISO,
               value: e.target.value,
             });
-            validatePetitionFromPaperSequence({ preventAutoScroll: true });
+            validatePetitionFromPaperSequence({ isSubmitting: true });
           }}
         />
         <FormGroup errorText={validationErrors.mailingDate}>
@@ -67,7 +67,7 @@ export const CaseInformation = connect(
             name="mailingDate"
             value={form.mailingDate || ''}
             onBlur={() =>
-              validatePetitionFromPaperSequence({ preventAutoScroll: true })
+              validatePetitionFromPaperSequence({ isSubmitting: true })
             }
             onChange={e => {
               updateFormValueSequence({
@@ -87,7 +87,7 @@ export const CaseInformation = connect(
             name="caseCaption"
             value={form.caseCaption}
             onBlur={() => {
-              validatePetitionFromPaperSequence({ preventAutoScroll: true });
+              validatePetitionFromPaperSequence({ isSubmitting: true });
             }}
             onChange={e => {
               updateFormValueSequence({
@@ -109,7 +109,7 @@ export const CaseInformation = connect(
               value: e.target.value,
             });
             clearPreferredTrialCitySequence();
-            validatePetitionFromPaperSequence({ preventAutoScroll: true });
+            validatePetitionFromPaperSequence({ isSubmitting: true });
           }}
         />
         <FormGroup>
@@ -150,7 +150,7 @@ export const CaseInformation = connect(
               key: e.target.name,
               value: e.target.value || null,
             });
-            validatePetitionFromPaperSequence({ preventAutoScroll: true });
+            validatePetitionFromPaperSequence({ isSubmitting: true });
           }}
         />
         <FormGroup errorText={validationErrors['object.missing']}>
@@ -167,7 +167,7 @@ export const CaseInformation = connect(
                   key: e.target.name,
                   value: e.target.checked,
                 });
-                validatePetitionFromPaperSequence({ preventAutoScroll: true });
+                validatePetitionFromPaperSequence({ isSubmitting: true });
               }}
             />
             <label
diff --git a/web-client/src/views/StartCaseInternal/Parties.tsx b/web-client/src/views/StartCaseInternal/Parties.tsx
index ac30f1a3726..7345ed47ec9 100644
--- a/web-client/src/views/StartCaseInternal/Parties.tsx
+++ b/web-client/src/views/StartCaseInternal/Parties.tsx
@@ -40,7 +40,7 @@ export const Parties = connect(
                 key: e.target.name,
                 value: e.target.value,
               });
-              validatePetitionFromPaperSequence({ preventAutoScroll: true });
+              validatePetitionFromPaperSequence({ isSubmitting: true });
             }}
           >
             <option value="">- Select -</option>
@@ -93,7 +93,7 @@ export const Parties = connect(
               }
               useSameAsPrimary={true}
               onBlur={() =>
-                validatePetitionFromPaperSequence({ preventAutoScroll: true })
+                validatePetitionFromPaperSequence({ isSubmitting: true })
               }
               onChange="updateFormValueAndCaseCaptionSequence"
             />
diff --git a/web-client/src/views/StartCaseInternal/StatisticsForm.tsx b/web-client/src/views/StartCaseInternal/StatisticsForm.tsx
index 1cc91db8c2b..755eedcedd5 100644
--- a/web-client/src/views/StartCaseInternal/StatisticsForm.tsx
+++ b/web-client/src/views/StartCaseInternal/StatisticsForm.tsx
@@ -50,7 +50,7 @@ export const StatisticsForm = connect(
           name={`statistics.${index}.irsDeficiencyAmount`}
           value={form.statistics[index].irsDeficiencyAmount || ''}
           onBlur={() =>
-            validatePetitionFromPaperSequence({ preventAutoScroll: true })
+            validatePetitionFromPaperSequence({ isSubmitting: true })
           }
           onValueChange={values => {
             updateStatisticsFormValueSequence({
@@ -136,7 +136,7 @@ export const StatisticsForm = connect(
                     value={form.statistics[index].year || ''}
                     onBlur={() =>
                       validatePetitionFromPaperSequence({
-                        preventAutoScroll: true,
+                        isSubmitting: true,
                       })
                     }
                     onChange={e => {
diff --git a/web-client/src/views/TrialSessions/SessionInformationForm.tsx b/web-client/src/views/TrialSessions/SessionInformationForm.tsx
index e556aa7eeb3..9848205f842 100644
--- a/web-client/src/views/TrialSessions/SessionInformationForm.tsx
+++ b/web-client/src/views/TrialSessions/SessionInformationForm.tsx
@@ -58,7 +58,7 @@ export const SessionInformationForm = connect<
                     type="radio"
                     value={value}
                     onBlur={() => {
-                      validateTrialSessionSequence();
+                      validateTrialSessionSequence({ isSubmitting: true });
                     }}
                     onChange={e => {
                       updateTrialSessionFormDataSequence({
@@ -100,7 +100,7 @@ export const SessionInformationForm = connect<
                     toFormat: DATE_FORMATS.ISO,
                     value: e.target.value,
                   });
-                  validateTrialSessionSequence();
+                  validateTrialSessionSequence({ isSubmitting: true });
                 }}
               />
             </div>
@@ -167,7 +167,9 @@ export const SessionInformationForm = connect<
                                 type="radio"
                                 value={option}
                                 onBlur={() => {
-                                  validateTrialSessionSequence();
+                                  validateTrialSessionSequence({
+                                    isSubmitting: true,
+                                  });
                                 }}
                                 onChange={e => {
                                   updateTrialSessionFormDataSequence({
@@ -215,7 +217,7 @@ export const SessionInformationForm = connect<
                     toFormat: DATE_FORMATS.ISO,
                     value: e.target.value,
                   });
-                  validateTrialSessionSequence();
+                  validateTrialSessionSequence({ isSubmitting: true });
                 }}
               />
             </div>
@@ -270,7 +272,9 @@ export const SessionInformationForm = connect<
                           key: e.target.name,
                           value: e.target.value,
                         });
-                        validateTrialSessionSequence();
+                        validateTrialSessionSequence({
+                          isSubmitting: true,
+                        });
                       }}
                     >
                       <option value="">- Select -</option>
@@ -312,7 +316,7 @@ export const SessionInformationForm = connect<
                         key: e.target.name,
                         value: e.target.value,
                       });
-                      validateTrialSessionSequence();
+                      validateTrialSessionSequence({ isSubmitting: true });
                     }}
                   />
                   <label
@@ -340,7 +344,7 @@ export const SessionInformationForm = connect<
                 type="text"
                 value={form.maxCases || ''}
                 onBlur={() => {
-                  validateTrialSessionSequence();
+                  validateTrialSessionSequence({ isSubmitting: true });
                 }}
                 onChange={e => {
                   updateTrialSessionFormDataSequence({
diff --git a/web-client/src/views/UserContactEditForm.tsx b/web-client/src/views/UserContactEditForm.tsx
index b9254ab226f..d8c4a88fd74 100644
--- a/web-client/src/views/UserContactEditForm.tsx
+++ b/web-client/src/views/UserContactEditForm.tsx
@@ -30,9 +30,7 @@ export const UserContactEditForm = connect(
         <Country
           bind={bind}
           type={type}
-          onBlur={() =>
-            validateUserContactSequence({ preventAutoScroll: true })
-          }
+          onBlur={() => validateUserContactSequence({ isSubmitting: true })}
           onChange={'updateFormValueSequence'}
           onChangeCountryType={'changeCountryTypeSequence'}
         />
@@ -40,18 +38,14 @@ export const UserContactEditForm = connect(
           <Address
             bind={bind}
             type={type}
-            onBlur={() =>
-              validateUserContactSequence({ preventAutoScroll: true })
-            }
+            onBlur={() => validateUserContactSequence({ isSubmitting: true })}
             onChange={'updateFormValueSequence'}
           />
         ) : (
           <InternationalAddress
             bind={bind}
             type={type}
-            onBlur={() =>
-              validateUserContactSequence({ preventAutoScroll: true })
-            }
+            onBlur={() => validateUserContactSequence({ isSubmitting: true })}
             onChange={'updateFormValueSequence'}
           />
         )}
@@ -70,9 +64,7 @@ export const UserContactEditForm = connect(
             name="contact.phone"
             type="text"
             value={form.contact.phone || ''}
-            onBlur={() =>
-              validateUserContactSequence({ preventAutoScroll: true })
-            }
+            onBlur={() => validateUserContactSequence({ isSubmitting: true })}
             onChange={e => {
               updateFormValueSequence({
                 key: e.target.name,

From ec8583b3fd40770b2f3b4bc7f44c1b3c35974515 Mon Sep 17 00:00:00 2001
From: Christopher Bisom <cbisom@flexion.us>
Date: Thu, 19 Sep 2024 18:05:20 -0400
Subject: [PATCH 11/33] 10074-bug: fix rename issue

---
 .../journey/petitionsClerkUpdatesCaseDetail.ts     |  2 +-
 .../actions/setValidationAlertErrorsAction.ts      |  2 +-
 .../AddPetitionerToCase/AddPetitionerToCase.tsx    |  2 +-
 .../src/views/CaseDetail/PetitionPaymentForm.tsx   | 12 ++++++------
 .../src/views/CaseDetailEdit/PartyInformation.tsx  |  2 +-
 .../views/EditPetitionerInformationInternal.tsx    |  2 +-
 web-client/src/views/IRSNotice.tsx                 |  2 +-
 .../Practitioners/PractitionerContactForm.tsx      |  8 ++++----
 web-client/src/views/StartCase/CaseTypeSelect.tsx  |  2 +-
 .../StartCaseInternal/CalculatePenaltiesModal.tsx  |  2 +-
 .../views/StartCaseInternal/CaseInformation.tsx    | 12 ++++++------
 web-client/src/views/StartCaseInternal/Parties.tsx |  4 ++--
 .../src/views/StartCaseInternal/StatisticsForm.tsx |  4 ++--
 .../views/TrialSessions/SessionInformationForm.tsx | 14 +++++++-------
 web-client/src/views/UserContactEditForm.tsx       |  8 ++++----
 15 files changed, 39 insertions(+), 39 deletions(-)

diff --git a/web-client/integration-tests/journey/petitionsClerkUpdatesCaseDetail.ts b/web-client/integration-tests/journey/petitionsClerkUpdatesCaseDetail.ts
index 566504508da..2b630a35a9d 100644
--- a/web-client/integration-tests/journey/petitionsClerkUpdatesCaseDetail.ts
+++ b/web-client/integration-tests/journey/petitionsClerkUpdatesCaseDetail.ts
@@ -142,8 +142,8 @@ export const petitionsClerkUpdatesCaseDetail = cerebralTest => {
       procedureType: 'Select a case procedure',
     });
     expect(cerebralTest.getState('alertError')).toEqual({
-      isSubmitting: false,
       messages: ['Select a case type', 'Select a case procedure'],
+      preventAutoScroll: false,
       title: 'Please correct the following errors on the page:',
     });
 
diff --git a/web-client/src/presenter/actions/setValidationAlertErrorsAction.ts b/web-client/src/presenter/actions/setValidationAlertErrorsAction.ts
index 8a75f62ef44..7a29ddefd26 100644
--- a/web-client/src/presenter/actions/setValidationAlertErrorsAction.ts
+++ b/web-client/src/presenter/actions/setValidationAlertErrorsAction.ts
@@ -86,7 +86,7 @@ export const setValidationAlertErrorsAction = ({
           }
         }),
     ),
-    preventAutoScroll: props.isSubmitting || false,
+    preventAutoScroll: !props.isSubmitting,
     title: 'Please correct the following errors on the page:',
   };
   store.set(state.alertError, alertError);
diff --git a/web-client/src/views/AddPetitionerToCase/AddPetitionerToCase.tsx b/web-client/src/views/AddPetitionerToCase/AddPetitionerToCase.tsx
index 5d5cda96b62..fda9b1b3433 100644
--- a/web-client/src/views/AddPetitionerToCase/AddPetitionerToCase.tsx
+++ b/web-client/src/views/AddPetitionerToCase/AddPetitionerToCase.tsx
@@ -51,7 +51,7 @@ export const AddPetitionerToCase = connect(
   }) {
     const type = 'contact';
     const bind = 'form';
-    const onBlur = () => validateAddPetitionerSequence({ isSubmitting: true });
+    const onBlur = () => validateAddPetitionerSequence({ isSubmitting: false });
 
     return (
       <>
diff --git a/web-client/src/views/CaseDetail/PetitionPaymentForm.tsx b/web-client/src/views/CaseDetail/PetitionPaymentForm.tsx
index 36a63ed2808..afe6c570781 100644
--- a/web-client/src/views/CaseDetail/PetitionPaymentForm.tsx
+++ b/web-client/src/views/CaseDetail/PetitionPaymentForm.tsx
@@ -44,7 +44,7 @@ const PetitionPaymentFormComponent: React.FC<ComponentProps & any> = ({
                   key: e.target.name,
                   value: e.target.value,
                 });
-                validateFormData({ isSubmitting: true });
+                validateFormData({ isSubmitting: false });
               }}
             />
             <label
@@ -70,7 +70,7 @@ const PetitionPaymentFormComponent: React.FC<ComponentProps & any> = ({
                   key: e.target.name,
                   value: e.target.value,
                 });
-                validateFormData({ isSubmitting: true });
+                validateFormData({ isSubmitting: false });
               }}
             />
             <label
@@ -96,7 +96,7 @@ const PetitionPaymentFormComponent: React.FC<ComponentProps & any> = ({
                   key: e.target.name,
                   value: e.target.value,
                 });
-                validateFormData({ isSubmitting: true });
+                validateFormData({ isSubmitting: false });
               }}
             />
             <label
@@ -123,7 +123,7 @@ const PetitionPaymentFormComponent: React.FC<ComponentProps & any> = ({
                 toFormat: DATE_FORMATS.ISO,
                 value: e.target.value,
               });
-              validateFormData({ isSubmitting: true });
+              validateFormData({ isSubmitting: false });
             }}
           />
 
@@ -139,7 +139,7 @@ const PetitionPaymentFormComponent: React.FC<ComponentProps & any> = ({
               id="petition-payment-method"
               name="petitionPaymentMethod"
               value={bind.petitionPaymentMethod || ''}
-              onBlur={() => validateFormData({ isSubmitting: true })}
+              onBlur={() => validateFormData({ isSubmitting: false })}
               onChange={e => {
                 onUpdate({
                   allowEmptyString: true,
@@ -164,7 +164,7 @@ const PetitionPaymentFormComponent: React.FC<ComponentProps & any> = ({
               toFormat: DATE_FORMATS.ISO,
               value: e.target.value,
             });
-            validateFormData({ isSubmitting: true });
+            validateFormData({ isSubmitting: false });
           }}
         />
       )}
diff --git a/web-client/src/views/CaseDetailEdit/PartyInformation.tsx b/web-client/src/views/CaseDetailEdit/PartyInformation.tsx
index 14528640f47..de9392e8a0f 100644
--- a/web-client/src/views/CaseDetailEdit/PartyInformation.tsx
+++ b/web-client/src/views/CaseDetailEdit/PartyInformation.tsx
@@ -89,7 +89,7 @@ export const PartyInformation = connect(
               showPrimaryContact={caseDetailEditHelper.showPrimaryContact}
               showSecondaryContact={caseDetailEditHelper.showSecondaryContact}
               useSameAsPrimary={true}
-              onBlur={() => validateCaseDetailSequence({ isSubmitting: true })}
+              onBlur={() => validateCaseDetailSequence({ isSubmitting: false })}
               onChange="updateFormValueAndCaseCaptionSequence"
             />
           </div>
diff --git a/web-client/src/views/EditPetitionerInformationInternal.tsx b/web-client/src/views/EditPetitionerInformationInternal.tsx
index 9829e5e2364..d66a4adf25d 100644
--- a/web-client/src/views/EditPetitionerInformationInternal.tsx
+++ b/web-client/src/views/EditPetitionerInformationInternal.tsx
@@ -55,7 +55,7 @@ export const EditPetitionerInformationInternal = connect(
   }) {
     const type = 'contact';
     const bind = 'form';
-    const onBlur = () => validatePetitionerSequence({ isSubmitting: true });
+    const onBlur = () => validatePetitionerSequence({ isSubmitting: false });
 
     return (
       <>
diff --git a/web-client/src/views/IRSNotice.tsx b/web-client/src/views/IRSNotice.tsx
index 37059f2c97d..0d1c0c94454 100644
--- a/web-client/src/views/IRSNotice.tsx
+++ b/web-client/src/views/IRSNotice.tsx
@@ -123,7 +123,7 @@ export const IRSNotice = connect<IrsNoticeType, typeof irsNoticeDependencies>(
               toFormat: constants.DATE_FORMATS.ISO,
               value: e.target.value,
             });
-            validateFormData({ isSubmitting: true });
+            validateFormData({ isSubmitting: false });
           }}
         />
       );
diff --git a/web-client/src/views/Practitioners/PractitionerContactForm.tsx b/web-client/src/views/Practitioners/PractitionerContactForm.tsx
index 4282a54d3e2..0b5eba105e3 100644
--- a/web-client/src/views/Practitioners/PractitionerContactForm.tsx
+++ b/web-client/src/views/Practitioners/PractitionerContactForm.tsx
@@ -33,7 +33,7 @@ export const PractitionerContactForm = connect(
         <Country
           bind={bind}
           type={type}
-          onBlur={() => onBlurValidationSequence({ isSubmitting: true })}
+          onBlur={() => onBlurValidationSequence({ isSubmitting: false })}
           onChange={onChangeSequenceName}
           onChangeCountryType={changeCountryTypeSequence}
         />
@@ -41,14 +41,14 @@ export const PractitionerContactForm = connect(
           <Address
             bind={bind}
             type={type}
-            onBlur={() => onBlurValidationSequence({ isSubmitting: true })}
+            onBlur={() => onBlurValidationSequence({ isSubmitting: false })}
             onChange={onChangeSequenceName}
           />
         ) : (
           <InternationalAddress
             bind={bind}
             type={type}
-            onBlur={() => onBlurValidationSequence({ isSubmitting: true })}
+            onBlur={() => onBlurValidationSequence({ isSubmitting: false })}
             onChange={onChangeSequenceName}
           />
         )}
@@ -69,7 +69,7 @@ export const PractitionerContactForm = connect(
                 name="contact.phone"
                 type="text"
                 value={form.contact.phone || ''}
-                onBlur={() => onBlurValidationSequence({ isSubmitting: true })}
+                onBlur={() => onBlurValidationSequence({ isSubmitting: false })}
                 onChange={e => {
                   onChangeUpdateSequence({
                     key: e.target.name,
diff --git a/web-client/src/views/StartCase/CaseTypeSelect.tsx b/web-client/src/views/StartCase/CaseTypeSelect.tsx
index 59c7af1b446..140c3c575e6 100644
--- a/web-client/src/views/StartCase/CaseTypeSelect.tsx
+++ b/web-client/src/views/StartCase/CaseTypeSelect.tsx
@@ -79,7 +79,7 @@ export const CaseTypeSelect = connect<
                   value: e.target.value,
                 });
                 if (onChangePreValidation) onChangePreValidation();
-                if (validateFormData) validateFormData({ isSubmitting: true });
+                if (validateFormData) validateFormData({ isSubmitting: false });
               }}
             >
               {allowDefaultOption && <option value="">-- Select --</option>}
diff --git a/web-client/src/views/StartCaseInternal/CalculatePenaltiesModal.tsx b/web-client/src/views/StartCaseInternal/CalculatePenaltiesModal.tsx
index 2e41d11c153..60b4b9c3763 100644
--- a/web-client/src/views/StartCaseInternal/CalculatePenaltiesModal.tsx
+++ b/web-client/src/views/StartCaseInternal/CalculatePenaltiesModal.tsx
@@ -40,7 +40,7 @@ export const CalculatePenaltiesModal = connect(
         cancelLabel="Cancel"
         cancelSequence={cancelSequence}
         confirmLabel="Calculate and Save"
-        confirmSequence={() => confirmSequence({ isSubmitting: true })}
+        confirmSequence={() => confirmSequence({ isSubmitting: false })}
         title={title}
       >
         {errors &&
diff --git a/web-client/src/views/StartCaseInternal/CaseInformation.tsx b/web-client/src/views/StartCaseInternal/CaseInformation.tsx
index 690791c531b..bc3f6513d17 100644
--- a/web-client/src/views/StartCaseInternal/CaseInformation.tsx
+++ b/web-client/src/views/StartCaseInternal/CaseInformation.tsx
@@ -53,7 +53,7 @@ export const CaseInformation = connect(
               toFormat: DATE_FORMATS.ISO,
               value: e.target.value,
             });
-            validatePetitionFromPaperSequence({ isSubmitting: true });
+            validatePetitionFromPaperSequence({ isSubmitting: false });
           }}
         />
         <FormGroup errorText={validationErrors.mailingDate}>
@@ -67,7 +67,7 @@ export const CaseInformation = connect(
             name="mailingDate"
             value={form.mailingDate || ''}
             onBlur={() =>
-              validatePetitionFromPaperSequence({ isSubmitting: true })
+              validatePetitionFromPaperSequence({ isSubmitting: false })
             }
             onChange={e => {
               updateFormValueSequence({
@@ -87,7 +87,7 @@ export const CaseInformation = connect(
             name="caseCaption"
             value={form.caseCaption}
             onBlur={() => {
-              validatePetitionFromPaperSequence({ isSubmitting: true });
+              validatePetitionFromPaperSequence({ isSubmitting: false });
             }}
             onChange={e => {
               updateFormValueSequence({
@@ -109,7 +109,7 @@ export const CaseInformation = connect(
               value: e.target.value,
             });
             clearPreferredTrialCitySequence();
-            validatePetitionFromPaperSequence({ isSubmitting: true });
+            validatePetitionFromPaperSequence({ isSubmitting: false });
           }}
         />
         <FormGroup>
@@ -150,7 +150,7 @@ export const CaseInformation = connect(
               key: e.target.name,
               value: e.target.value || null,
             });
-            validatePetitionFromPaperSequence({ isSubmitting: true });
+            validatePetitionFromPaperSequence({ isSubmitting: false });
           }}
         />
         <FormGroup errorText={validationErrors['object.missing']}>
@@ -167,7 +167,7 @@ export const CaseInformation = connect(
                   key: e.target.name,
                   value: e.target.checked,
                 });
-                validatePetitionFromPaperSequence({ isSubmitting: true });
+                validatePetitionFromPaperSequence({ isSubmitting: false });
               }}
             />
             <label
diff --git a/web-client/src/views/StartCaseInternal/Parties.tsx b/web-client/src/views/StartCaseInternal/Parties.tsx
index 7345ed47ec9..46b7dcb7f32 100644
--- a/web-client/src/views/StartCaseInternal/Parties.tsx
+++ b/web-client/src/views/StartCaseInternal/Parties.tsx
@@ -40,7 +40,7 @@ export const Parties = connect(
                 key: e.target.name,
                 value: e.target.value,
               });
-              validatePetitionFromPaperSequence({ isSubmitting: true });
+              validatePetitionFromPaperSequence({ isSubmitting: false });
             }}
           >
             <option value="">- Select -</option>
@@ -93,7 +93,7 @@ export const Parties = connect(
               }
               useSameAsPrimary={true}
               onBlur={() =>
-                validatePetitionFromPaperSequence({ isSubmitting: true })
+                validatePetitionFromPaperSequence({ isSubmitting: false })
               }
               onChange="updateFormValueAndCaseCaptionSequence"
             />
diff --git a/web-client/src/views/StartCaseInternal/StatisticsForm.tsx b/web-client/src/views/StartCaseInternal/StatisticsForm.tsx
index 755eedcedd5..f5598284934 100644
--- a/web-client/src/views/StartCaseInternal/StatisticsForm.tsx
+++ b/web-client/src/views/StartCaseInternal/StatisticsForm.tsx
@@ -50,7 +50,7 @@ export const StatisticsForm = connect(
           name={`statistics.${index}.irsDeficiencyAmount`}
           value={form.statistics[index].irsDeficiencyAmount || ''}
           onBlur={() =>
-            validatePetitionFromPaperSequence({ isSubmitting: true })
+            validatePetitionFromPaperSequence({ isSubmitting: false })
           }
           onValueChange={values => {
             updateStatisticsFormValueSequence({
@@ -136,7 +136,7 @@ export const StatisticsForm = connect(
                     value={form.statistics[index].year || ''}
                     onBlur={() =>
                       validatePetitionFromPaperSequence({
-                        isSubmitting: true,
+                        isSubmitting: false,
                       })
                     }
                     onChange={e => {
diff --git a/web-client/src/views/TrialSessions/SessionInformationForm.tsx b/web-client/src/views/TrialSessions/SessionInformationForm.tsx
index 9848205f842..a8f0a472db2 100644
--- a/web-client/src/views/TrialSessions/SessionInformationForm.tsx
+++ b/web-client/src/views/TrialSessions/SessionInformationForm.tsx
@@ -58,7 +58,7 @@ export const SessionInformationForm = connect<
                     type="radio"
                     value={value}
                     onBlur={() => {
-                      validateTrialSessionSequence({ isSubmitting: true });
+                      validateTrialSessionSequence({ isSubmitting: false });
                     }}
                     onChange={e => {
                       updateTrialSessionFormDataSequence({
@@ -100,7 +100,7 @@ export const SessionInformationForm = connect<
                     toFormat: DATE_FORMATS.ISO,
                     value: e.target.value,
                   });
-                  validateTrialSessionSequence({ isSubmitting: true });
+                  validateTrialSessionSequence({ isSubmitting: false });
                 }}
               />
             </div>
@@ -168,7 +168,7 @@ export const SessionInformationForm = connect<
                                 value={option}
                                 onBlur={() => {
                                   validateTrialSessionSequence({
-                                    isSubmitting: true,
+                                    isSubmitting: false,
                                   });
                                 }}
                                 onChange={e => {
@@ -217,7 +217,7 @@ export const SessionInformationForm = connect<
                     toFormat: DATE_FORMATS.ISO,
                     value: e.target.value,
                   });
-                  validateTrialSessionSequence({ isSubmitting: true });
+                  validateTrialSessionSequence({ isSubmitting: false });
                 }}
               />
             </div>
@@ -273,7 +273,7 @@ export const SessionInformationForm = connect<
                           value: e.target.value,
                         });
                         validateTrialSessionSequence({
-                          isSubmitting: true,
+                          isSubmitting: false,
                         });
                       }}
                     >
@@ -316,7 +316,7 @@ export const SessionInformationForm = connect<
                         key: e.target.name,
                         value: e.target.value,
                       });
-                      validateTrialSessionSequence({ isSubmitting: true });
+                      validateTrialSessionSequence({ isSubmitting: false });
                     }}
                   />
                   <label
@@ -344,7 +344,7 @@ export const SessionInformationForm = connect<
                 type="text"
                 value={form.maxCases || ''}
                 onBlur={() => {
-                  validateTrialSessionSequence({ isSubmitting: true });
+                  validateTrialSessionSequence({ isSubmitting: false });
                 }}
                 onChange={e => {
                   updateTrialSessionFormDataSequence({
diff --git a/web-client/src/views/UserContactEditForm.tsx b/web-client/src/views/UserContactEditForm.tsx
index d8c4a88fd74..3f5204b5b30 100644
--- a/web-client/src/views/UserContactEditForm.tsx
+++ b/web-client/src/views/UserContactEditForm.tsx
@@ -30,7 +30,7 @@ export const UserContactEditForm = connect(
         <Country
           bind={bind}
           type={type}
-          onBlur={() => validateUserContactSequence({ isSubmitting: true })}
+          onBlur={() => validateUserContactSequence({ isSubmitting: false })}
           onChange={'updateFormValueSequence'}
           onChangeCountryType={'changeCountryTypeSequence'}
         />
@@ -38,14 +38,14 @@ export const UserContactEditForm = connect(
           <Address
             bind={bind}
             type={type}
-            onBlur={() => validateUserContactSequence({ isSubmitting: true })}
+            onBlur={() => validateUserContactSequence({ isSubmitting: false })}
             onChange={'updateFormValueSequence'}
           />
         ) : (
           <InternationalAddress
             bind={bind}
             type={type}
-            onBlur={() => validateUserContactSequence({ isSubmitting: true })}
+            onBlur={() => validateUserContactSequence({ isSubmitting: false })}
             onChange={'updateFormValueSequence'}
           />
         )}
@@ -64,7 +64,7 @@ export const UserContactEditForm = connect(
             name="contact.phone"
             type="text"
             value={form.contact.phone || ''}
-            onBlur={() => validateUserContactSequence({ isSubmitting: true })}
+            onBlur={() => validateUserContactSequence({ isSubmitting: false })}
             onChange={e => {
               updateFormValueSequence({
                 key: e.target.name,

From eed3942f2667db614f43cec70864cef1ab37e72b Mon Sep 17 00:00:00 2001
From: Christopher Bisom <cbisom@flexion.us>
Date: Thu, 19 Sep 2024 18:47:41 -0400
Subject: [PATCH 12/33] 10075-bug: trying a different strategy to see if it is
 better

---
 .../setScrollToErrorNotificationAction.ts     |  6 ++++++
 .../actions/setValidationAlertErrorsAction.ts |  2 +-
 .../openConfirmPaperServiceModalSequence.ts   |  2 ++
 .../sequences/submitPaperFilingSequence.ts    |  4 ++--
 .../submitPetitionFromPaperSequence.ts        |  7 ++++++-
 .../sequences/submitTrialSessionSequence.ts   |  2 ++
 .../sequences/validateDocketEntrySequence.ts  |  3 ++-
 .../AddPetitionerToCase.tsx                   |  2 +-
 .../views/CaseDetail/PetitionPaymentForm.tsx  | 12 +++++------
 .../views/CaseDetailEdit/PartyInformation.tsx |  2 +-
 .../EditPetitionerInformationInternal.tsx     |  2 +-
 web-client/src/views/IRSNotice.tsx            |  2 +-
 .../src/views/PaperFiling/PaperFiling.tsx     |  3 +++
 .../Practitioners/PractitionerContactForm.tsx |  8 ++++----
 web-client/src/views/ScanBatchPreviewer.tsx   |  4 +++-
 .../src/views/StartCase/CaseTypeSelect.tsx    |  2 +-
 .../CalculatePenaltiesModal.tsx               |  2 +-
 .../StartCaseInternal/CaseInformation.tsx     | 14 ++++++-------
 .../src/views/StartCaseInternal/Parties.tsx   |  6 ++----
 .../StartCaseInternal/StatisticsForm.tsx      | 10 ++--------
 .../TrialSessions/SessionInformationForm.tsx  | 20 +++++++------------
 web-client/src/views/UserContactEditForm.tsx  |  8 ++++----
 22 files changed, 64 insertions(+), 59 deletions(-)
 create mode 100644 web-client/src/presenter/actions/setScrollToErrorNotificationAction.ts

diff --git a/web-client/src/presenter/actions/setScrollToErrorNotificationAction.ts b/web-client/src/presenter/actions/setScrollToErrorNotificationAction.ts
new file mode 100644
index 00000000000..b2dd69b93e0
--- /dev/null
+++ b/web-client/src/presenter/actions/setScrollToErrorNotificationAction.ts
@@ -0,0 +1,6 @@
+export const setScrollToErrorNotificationAction = ({ props }) => {
+  return {
+    ...props,
+    scrollToErrorNotification: true,
+  };
+};
diff --git a/web-client/src/presenter/actions/setValidationAlertErrorsAction.ts b/web-client/src/presenter/actions/setValidationAlertErrorsAction.ts
index 7a29ddefd26..7af5683692a 100644
--- a/web-client/src/presenter/actions/setValidationAlertErrorsAction.ts
+++ b/web-client/src/presenter/actions/setValidationAlertErrorsAction.ts
@@ -86,7 +86,7 @@ export const setValidationAlertErrorsAction = ({
           }
         }),
     ),
-    preventAutoScroll: !props.isSubmitting,
+    preventAutoScroll: !props.scrollToErrorNotification,
     title: 'Please correct the following errors on the page:',
   };
   store.set(state.alertError, alertError);
diff --git a/web-client/src/presenter/sequences/openConfirmPaperServiceModalSequence.ts b/web-client/src/presenter/sequences/openConfirmPaperServiceModalSequence.ts
index 4f54c367bbc..222f5295be0 100644
--- a/web-client/src/presenter/sequences/openConfirmPaperServiceModalSequence.ts
+++ b/web-client/src/presenter/sequences/openConfirmPaperServiceModalSequence.ts
@@ -5,6 +5,7 @@ import { isDocketEntryMultiDocketableAction } from '../actions/CaseConsolidation
 import { setDocumentIsRequiredAction } from '../actions/DocketEntry/setDocumentIsRequiredAction';
 import { setFilersFromFilersMapAction } from '../actions/setFilersFromFilersMapAction';
 import { setMultiDocketingCheckboxesAction } from '../actions/CaseConsolidation/setMultiDocketingCheckboxesAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
 import { setShowModalFactoryAction } from '../actions/setShowModalFactoryAction';
 import { setValidationAlertErrorsAction } from '../actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
@@ -24,6 +25,7 @@ export const openConfirmPaperServiceModalSequence = [
     error: [
       suggestSaveForLaterValidationAction,
       setValidationErrorsAction,
+      setScrollToErrorNotificationAction,
       setValidationAlertErrorsAction,
     ],
     success: [
diff --git a/web-client/src/presenter/sequences/submitPaperFilingSequence.ts b/web-client/src/presenter/sequences/submitPaperFilingSequence.ts
index 9e52df724ac..2e381abba87 100644
--- a/web-client/src/presenter/sequences/submitPaperFilingSequence.ts
+++ b/web-client/src/presenter/sequences/submitPaperFilingSequence.ts
@@ -8,10 +8,10 @@ import { getDocketNumbersForConsolidatedServiceAction } from '../actions/getDock
 import { isEditingDocketEntryAction } from '../actions/CourtIssuedDocketEntry/isEditingDocketEntryAction';
 import { isFileAttachedAction } from '../actions/isFileAttachedAction';
 import { isWorkItemAlreadyCompletedAction } from '../actions/isWorkItemAlreadyCompletedAction';
-import { setAlertErrorAction } from '../actions/setAlertErrorAction';
 import { setCaseAction } from '../actions/setCaseAction';
 import { setDocumentIsRequiredAction } from '../actions/DocketEntry/setDocumentIsRequiredAction';
 import { setFilersFromFilersMapAction } from '../actions/setFilersFromFilersMapAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
 import { setShowModalFactoryAction } from '../actions/setShowModalFactoryAction';
 import { setValidationAlertErrorsAction } from '../actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
@@ -38,8 +38,8 @@ export const submitPaperFilingSequence = [
       {
         error: [
           suggestSaveForLaterValidationAction,
-          setAlertErrorAction,
           setValidationErrorsAction,
+          setScrollToErrorNotificationAction,
           setValidationAlertErrorsAction,
         ],
         success: [
diff --git a/web-client/src/presenter/sequences/submitPetitionFromPaperSequence.ts b/web-client/src/presenter/sequences/submitPetitionFromPaperSequence.ts
index ae19336842c..ec6ebde4c8e 100644
--- a/web-client/src/presenter/sequences/submitPetitionFromPaperSequence.ts
+++ b/web-client/src/presenter/sequences/submitPetitionFromPaperSequence.ts
@@ -12,6 +12,7 @@ import { setCaseAction } from '../actions/setCaseAction';
 import { setCaseTypeAction } from '../actions/setCaseTypeAction';
 import { setDocketEntryIdAction } from '../actions/setDocketEntryIdAction';
 import { setProgressForFileUploadAction } from '@web-client/presenter/actions/setProgressForFileUploadAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
 import { setShowModalFactoryAction } from '../actions/setShowModalFactoryAction';
 import { setValidationAlertErrorsAction } from '../actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
@@ -32,7 +33,11 @@ export const submitPetitionFromPaperSequence = [
       filterEmptyStatisticsAction,
       validatePetitionFromPaperAction,
       {
-        error: [setValidationErrorsAction, setValidationAlertErrorsAction],
+        error: [
+          setValidationErrorsAction,
+          setScrollToErrorNotificationAction,
+          setValidationAlertErrorsAction,
+        ],
         success: [
           setupCurrentPageAction('Interstitial'),
           stopShowValidationAction,
diff --git a/web-client/src/presenter/sequences/submitTrialSessionSequence.ts b/web-client/src/presenter/sequences/submitTrialSessionSequence.ts
index ae9e5c64839..2168cbb6d8d 100644
--- a/web-client/src/presenter/sequences/submitTrialSessionSequence.ts
+++ b/web-client/src/presenter/sequences/submitTrialSessionSequence.ts
@@ -7,6 +7,7 @@ import { setActiveTrialSessionsTabAction } from '@web-client/presenter/actions/T
 import { setAlertErrorAction } from '../actions/setAlertErrorAction';
 import { setAlertSuccessAction } from '../actions/setAlertSuccessAction';
 import { setSaveAlertsForNavigationAction } from '../actions/setSaveAlertsForNavigationAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
 import { setValidationAlertErrorsAction } from '../actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
 import { showProgressSequenceDecorator } from '../utilities/showProgressSequenceDecorator';
@@ -23,6 +24,7 @@ export const submitTrialSessionSequence = [
     error: [
       setAlertErrorAction,
       setValidationErrorsAction,
+      setScrollToErrorNotificationAction,
       setValidationAlertErrorsAction,
     ],
     success: showProgressSequenceDecorator([
diff --git a/web-client/src/presenter/sequences/validateDocketEntrySequence.ts b/web-client/src/presenter/sequences/validateDocketEntrySequence.ts
index 53c2347636b..034cd1ea277 100644
--- a/web-client/src/presenter/sequences/validateDocketEntrySequence.ts
+++ b/web-client/src/presenter/sequences/validateDocketEntrySequence.ts
@@ -1,5 +1,6 @@
 import { clearAlertsAction } from '../actions/clearAlertsAction';
 import { setFilersFromFilersMapAction } from '../actions/setFilersFromFilersMapAction';
+import { setValidationAlertErrorsAction } from '@web-client/presenter/actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
 import { shouldValidateAction } from '../actions/shouldValidateAction';
 import { validateDocketEntryAction } from '../actions/DocketEntry/validateDocketEntryAction';
@@ -12,7 +13,7 @@ export const validateDocketEntrySequence = [
       setFilersFromFilersMapAction,
       validateDocketEntryAction,
       {
-        error: [setValidationErrorsAction],
+        error: [setValidationErrorsAction, setValidationAlertErrorsAction],
         success: [clearAlertsAction],
       },
     ],
diff --git a/web-client/src/views/AddPetitionerToCase/AddPetitionerToCase.tsx b/web-client/src/views/AddPetitionerToCase/AddPetitionerToCase.tsx
index fda9b1b3433..b1c1b817d87 100644
--- a/web-client/src/views/AddPetitionerToCase/AddPetitionerToCase.tsx
+++ b/web-client/src/views/AddPetitionerToCase/AddPetitionerToCase.tsx
@@ -51,7 +51,7 @@ export const AddPetitionerToCase = connect(
   }) {
     const type = 'contact';
     const bind = 'form';
-    const onBlur = () => validateAddPetitionerSequence({ isSubmitting: false });
+    const onBlur = validateAddPetitionerSequence;
 
     return (
       <>
diff --git a/web-client/src/views/CaseDetail/PetitionPaymentForm.tsx b/web-client/src/views/CaseDetail/PetitionPaymentForm.tsx
index afe6c570781..d913b5497c0 100644
--- a/web-client/src/views/CaseDetail/PetitionPaymentForm.tsx
+++ b/web-client/src/views/CaseDetail/PetitionPaymentForm.tsx
@@ -44,7 +44,7 @@ const PetitionPaymentFormComponent: React.FC<ComponentProps & any> = ({
                   key: e.target.name,
                   value: e.target.value,
                 });
-                validateFormData({ isSubmitting: false });
+                validateFormData();
               }}
             />
             <label
@@ -70,7 +70,7 @@ const PetitionPaymentFormComponent: React.FC<ComponentProps & any> = ({
                   key: e.target.name,
                   value: e.target.value,
                 });
-                validateFormData({ isSubmitting: false });
+                validateFormData();
               }}
             />
             <label
@@ -96,7 +96,7 @@ const PetitionPaymentFormComponent: React.FC<ComponentProps & any> = ({
                   key: e.target.name,
                   value: e.target.value,
                 });
-                validateFormData({ isSubmitting: false });
+                validateFormData();
               }}
             />
             <label
@@ -123,7 +123,7 @@ const PetitionPaymentFormComponent: React.FC<ComponentProps & any> = ({
                 toFormat: DATE_FORMATS.ISO,
                 value: e.target.value,
               });
-              validateFormData({ isSubmitting: false });
+              validateFormData();
             }}
           />
 
@@ -139,7 +139,7 @@ const PetitionPaymentFormComponent: React.FC<ComponentProps & any> = ({
               id="petition-payment-method"
               name="petitionPaymentMethod"
               value={bind.petitionPaymentMethod || ''}
-              onBlur={() => validateFormData({ isSubmitting: false })}
+              onBlur={validateFormData}
               onChange={e => {
                 onUpdate({
                   allowEmptyString: true,
@@ -164,7 +164,7 @@ const PetitionPaymentFormComponent: React.FC<ComponentProps & any> = ({
               toFormat: DATE_FORMATS.ISO,
               value: e.target.value,
             });
-            validateFormData({ isSubmitting: false });
+            validateFormData();
           }}
         />
       )}
diff --git a/web-client/src/views/CaseDetailEdit/PartyInformation.tsx b/web-client/src/views/CaseDetailEdit/PartyInformation.tsx
index de9392e8a0f..b498cdf7995 100644
--- a/web-client/src/views/CaseDetailEdit/PartyInformation.tsx
+++ b/web-client/src/views/CaseDetailEdit/PartyInformation.tsx
@@ -89,7 +89,7 @@ export const PartyInformation = connect(
               showPrimaryContact={caseDetailEditHelper.showPrimaryContact}
               showSecondaryContact={caseDetailEditHelper.showSecondaryContact}
               useSameAsPrimary={true}
-              onBlur={() => validateCaseDetailSequence({ isSubmitting: false })}
+              onBlur={validateCaseDetailSequence}
               onChange="updateFormValueAndCaseCaptionSequence"
             />
           </div>
diff --git a/web-client/src/views/EditPetitionerInformationInternal.tsx b/web-client/src/views/EditPetitionerInformationInternal.tsx
index d66a4adf25d..a54da707cbd 100644
--- a/web-client/src/views/EditPetitionerInformationInternal.tsx
+++ b/web-client/src/views/EditPetitionerInformationInternal.tsx
@@ -55,7 +55,7 @@ export const EditPetitionerInformationInternal = connect(
   }) {
     const type = 'contact';
     const bind = 'form';
-    const onBlur = () => validatePetitionerSequence({ isSubmitting: false });
+    const onBlur = validatePetitionerSequence;
 
     return (
       <>
diff --git a/web-client/src/views/IRSNotice.tsx b/web-client/src/views/IRSNotice.tsx
index 0d1c0c94454..261d1b4de5c 100644
--- a/web-client/src/views/IRSNotice.tsx
+++ b/web-client/src/views/IRSNotice.tsx
@@ -123,7 +123,7 @@ export const IRSNotice = connect<IrsNoticeType, typeof irsNoticeDependencies>(
               toFormat: constants.DATE_FORMATS.ISO,
               value: e.target.value,
             });
-            validateFormData({ isSubmitting: false });
+            validateFormData();
           }}
         />
       );
diff --git a/web-client/src/views/PaperFiling/PaperFiling.tsx b/web-client/src/views/PaperFiling/PaperFiling.tsx
index 8d51bf695a5..163a6178b63 100644
--- a/web-client/src/views/PaperFiling/PaperFiling.tsx
+++ b/web-client/src/views/PaperFiling/PaperFiling.tsx
@@ -29,6 +29,7 @@ export const PaperFiling = connect(
     paperDocketEntryHelper: state.paperDocketEntryHelper,
     showModal: state.modal.showModal,
     submitPaperFilingSequence: sequences.submitPaperFilingSequence,
+    validateDocketEntrySequence: sequences.validateDocketEntrySequence,
   },
   function PaperFiling({
     closeModalAndReturnToCaseDetailSequence,
@@ -40,6 +41,7 @@ export const PaperFiling = connect(
     paperDocketEntryHelper,
     showModal,
     submitPaperFilingSequence,
+    validateDocketEntrySequence,
   }) {
     return (
       <>
@@ -132,6 +134,7 @@ export const PaperFiling = connect(
               <ScanBatchPreviewer
                 documentType="primaryDocumentFile"
                 title="Add Document"
+                validateSequence={validateDocketEntrySequence}
               />
             </div>
           </div>
diff --git a/web-client/src/views/Practitioners/PractitionerContactForm.tsx b/web-client/src/views/Practitioners/PractitionerContactForm.tsx
index 0b5eba105e3..93617163333 100644
--- a/web-client/src/views/Practitioners/PractitionerContactForm.tsx
+++ b/web-client/src/views/Practitioners/PractitionerContactForm.tsx
@@ -33,7 +33,7 @@ export const PractitionerContactForm = connect(
         <Country
           bind={bind}
           type={type}
-          onBlur={() => onBlurValidationSequence({ isSubmitting: false })}
+          onBlur={onBlurValidationSequence}
           onChange={onChangeSequenceName}
           onChangeCountryType={changeCountryTypeSequence}
         />
@@ -41,14 +41,14 @@ export const PractitionerContactForm = connect(
           <Address
             bind={bind}
             type={type}
-            onBlur={() => onBlurValidationSequence({ isSubmitting: false })}
+            onBlur={onBlurValidationSequence}
             onChange={onChangeSequenceName}
           />
         ) : (
           <InternationalAddress
             bind={bind}
             type={type}
-            onBlur={() => onBlurValidationSequence({ isSubmitting: false })}
+            onBlur={onBlurValidationSequence}
             onChange={onChangeSequenceName}
           />
         )}
@@ -69,7 +69,7 @@ export const PractitionerContactForm = connect(
                 name="contact.phone"
                 type="text"
                 value={form.contact.phone || ''}
-                onBlur={() => onBlurValidationSequence({ isSubmitting: false })}
+                onBlur={onBlurValidationSequence}
                 onChange={e => {
                   onChangeUpdateSequence({
                     key: e.target.name,
diff --git a/web-client/src/views/ScanBatchPreviewer.tsx b/web-client/src/views/ScanBatchPreviewer.tsx
index f583f2640cb..c6db9b375c9 100644
--- a/web-client/src/views/ScanBatchPreviewer.tsx
+++ b/web-client/src/views/ScanBatchPreviewer.tsx
@@ -280,7 +280,9 @@ export const ScanBatchPreviewer = connect(
         <>
           {showModal === 'ConfirmDeletePDFModal' && (
             <ConfirmDeletePDFModal
-              confirmSequence={removeScannedPdfSequence}
+              confirmSequence={() => {
+                removeScannedPdfSequence();
+              }}
               confirmText="Yes, Delete"
               modalContent="This action cannot be undone."
               title="Are you sure you want to delete this PDF?"
diff --git a/web-client/src/views/StartCase/CaseTypeSelect.tsx b/web-client/src/views/StartCase/CaseTypeSelect.tsx
index 140c3c575e6..dd60e3641f1 100644
--- a/web-client/src/views/StartCase/CaseTypeSelect.tsx
+++ b/web-client/src/views/StartCase/CaseTypeSelect.tsx
@@ -79,7 +79,7 @@ export const CaseTypeSelect = connect<
                   value: e.target.value,
                 });
                 if (onChangePreValidation) onChangePreValidation();
-                if (validateFormData) validateFormData({ isSubmitting: false });
+                if (validateFormData) validateFormData();
               }}
             >
               {allowDefaultOption && <option value="">-- Select --</option>}
diff --git a/web-client/src/views/StartCaseInternal/CalculatePenaltiesModal.tsx b/web-client/src/views/StartCaseInternal/CalculatePenaltiesModal.tsx
index 60b4b9c3763..ba43b63e5e9 100644
--- a/web-client/src/views/StartCaseInternal/CalculatePenaltiesModal.tsx
+++ b/web-client/src/views/StartCaseInternal/CalculatePenaltiesModal.tsx
@@ -40,7 +40,7 @@ export const CalculatePenaltiesModal = connect(
         cancelLabel="Cancel"
         cancelSequence={cancelSequence}
         confirmLabel="Calculate and Save"
-        confirmSequence={() => confirmSequence({ isSubmitting: false })}
+        confirmSequence={confirmSequence}
         title={title}
       >
         {errors &&
diff --git a/web-client/src/views/StartCaseInternal/CaseInformation.tsx b/web-client/src/views/StartCaseInternal/CaseInformation.tsx
index bc3f6513d17..9014db18232 100644
--- a/web-client/src/views/StartCaseInternal/CaseInformation.tsx
+++ b/web-client/src/views/StartCaseInternal/CaseInformation.tsx
@@ -53,7 +53,7 @@ export const CaseInformation = connect(
               toFormat: DATE_FORMATS.ISO,
               value: e.target.value,
             });
-            validatePetitionFromPaperSequence({ isSubmitting: false });
+            validatePetitionFromPaperSequence();
           }}
         />
         <FormGroup errorText={validationErrors.mailingDate}>
@@ -66,9 +66,7 @@ export const CaseInformation = connect(
             maxLength="25"
             name="mailingDate"
             value={form.mailingDate || ''}
-            onBlur={() =>
-              validatePetitionFromPaperSequence({ isSubmitting: false })
-            }
+            onBlur={() => validatePetitionFromPaperSequence()}
             onChange={e => {
               updateFormValueSequence({
                 key: e.target.name,
@@ -87,7 +85,7 @@ export const CaseInformation = connect(
             name="caseCaption"
             value={form.caseCaption}
             onBlur={() => {
-              validatePetitionFromPaperSequence({ isSubmitting: false });
+              validatePetitionFromPaperSequence();
             }}
             onChange={e => {
               updateFormValueSequence({
@@ -109,7 +107,7 @@ export const CaseInformation = connect(
               value: e.target.value,
             });
             clearPreferredTrialCitySequence();
-            validatePetitionFromPaperSequence({ isSubmitting: false });
+            validatePetitionFromPaperSequence();
           }}
         />
         <FormGroup>
@@ -150,7 +148,7 @@ export const CaseInformation = connect(
               key: e.target.name,
               value: e.target.value || null,
             });
-            validatePetitionFromPaperSequence({ isSubmitting: false });
+            validatePetitionFromPaperSequence();
           }}
         />
         <FormGroup errorText={validationErrors['object.missing']}>
@@ -167,7 +165,7 @@ export const CaseInformation = connect(
                   key: e.target.name,
                   value: e.target.checked,
                 });
-                validatePetitionFromPaperSequence({ isSubmitting: false });
+                validatePetitionFromPaperSequence();
               }}
             />
             <label
diff --git a/web-client/src/views/StartCaseInternal/Parties.tsx b/web-client/src/views/StartCaseInternal/Parties.tsx
index 46b7dcb7f32..5e36e2f1df4 100644
--- a/web-client/src/views/StartCaseInternal/Parties.tsx
+++ b/web-client/src/views/StartCaseInternal/Parties.tsx
@@ -40,7 +40,7 @@ export const Parties = connect(
                 key: e.target.name,
                 value: e.target.value,
               });
-              validatePetitionFromPaperSequence({ isSubmitting: false });
+              validatePetitionFromPaperSequence();
             }}
           >
             <option value="">- Select -</option>
@@ -92,9 +92,7 @@ export const Parties = connect(
                 startCaseInternalHelper.showSecondaryContact
               }
               useSameAsPrimary={true}
-              onBlur={() =>
-                validatePetitionFromPaperSequence({ isSubmitting: false })
-              }
+              onBlur={validatePetitionFromPaperSequence}
               onChange="updateFormValueAndCaseCaptionSequence"
             />
           </div>
diff --git a/web-client/src/views/StartCaseInternal/StatisticsForm.tsx b/web-client/src/views/StartCaseInternal/StatisticsForm.tsx
index f5598284934..f7209ee7e84 100644
--- a/web-client/src/views/StartCaseInternal/StatisticsForm.tsx
+++ b/web-client/src/views/StartCaseInternal/StatisticsForm.tsx
@@ -49,9 +49,7 @@ export const StatisticsForm = connect(
           id={`deficiency-amount-${index}`}
           name={`statistics.${index}.irsDeficiencyAmount`}
           value={form.statistics[index].irsDeficiencyAmount || ''}
-          onBlur={() =>
-            validatePetitionFromPaperSequence({ isSubmitting: false })
-          }
+          onBlur={validatePetitionFromPaperSequence}
           onValueChange={values => {
             updateStatisticsFormValueSequence({
               key: `statistics.${index}.irsDeficiencyAmount`,
@@ -134,11 +132,7 @@ export const StatisticsForm = connect(
                     name={`statistics.${index}.year`}
                     placeholder="YYYY"
                     value={form.statistics[index].year || ''}
-                    onBlur={() =>
-                      validatePetitionFromPaperSequence({
-                        isSubmitting: false,
-                      })
-                    }
+                    onBlur={() => validatePetitionFromPaperSequence()}
                     onChange={e => {
                       updateStatisticsFormValueSequence({
                         key: e.target.name,
diff --git a/web-client/src/views/TrialSessions/SessionInformationForm.tsx b/web-client/src/views/TrialSessions/SessionInformationForm.tsx
index a8f0a472db2..cd76e2a918d 100644
--- a/web-client/src/views/TrialSessions/SessionInformationForm.tsx
+++ b/web-client/src/views/TrialSessions/SessionInformationForm.tsx
@@ -57,9 +57,7 @@ export const SessionInformationForm = connect<
                     name="sessionScope"
                     type="radio"
                     value={value}
-                    onBlur={() => {
-                      validateTrialSessionSequence({ isSubmitting: false });
-                    }}
+                    onBlur={() => validateTrialSessionSequence()}
                     onChange={e => {
                       updateTrialSessionFormDataSequence({
                         key: e.target.name,
@@ -100,7 +98,7 @@ export const SessionInformationForm = connect<
                     toFormat: DATE_FORMATS.ISO,
                     value: e.target.value,
                   });
-                  validateTrialSessionSequence({ isSubmitting: false });
+                  validateTrialSessionSequence();
                 }}
               />
             </div>
@@ -167,9 +165,7 @@ export const SessionInformationForm = connect<
                                 type="radio"
                                 value={option}
                                 onBlur={() => {
-                                  validateTrialSessionSequence({
-                                    isSubmitting: false,
-                                  });
+                                  validateTrialSessionSequence();
                                 }}
                                 onChange={e => {
                                   updateTrialSessionFormDataSequence({
@@ -217,7 +213,7 @@ export const SessionInformationForm = connect<
                     toFormat: DATE_FORMATS.ISO,
                     value: e.target.value,
                   });
-                  validateTrialSessionSequence({ isSubmitting: false });
+                  validateTrialSessionSequence();
                 }}
               />
             </div>
@@ -272,9 +268,7 @@ export const SessionInformationForm = connect<
                           key: e.target.name,
                           value: e.target.value,
                         });
-                        validateTrialSessionSequence({
-                          isSubmitting: false,
-                        });
+                        validateTrialSessionSequence();
                       }}
                     >
                       <option value="">- Select -</option>
@@ -316,7 +310,7 @@ export const SessionInformationForm = connect<
                         key: e.target.name,
                         value: e.target.value,
                       });
-                      validateTrialSessionSequence({ isSubmitting: false });
+                      validateTrialSessionSequence();
                     }}
                   />
                   <label
@@ -344,7 +338,7 @@ export const SessionInformationForm = connect<
                 type="text"
                 value={form.maxCases || ''}
                 onBlur={() => {
-                  validateTrialSessionSequence({ isSubmitting: false });
+                  validateTrialSessionSequence();
                 }}
                 onChange={e => {
                   updateTrialSessionFormDataSequence({
diff --git a/web-client/src/views/UserContactEditForm.tsx b/web-client/src/views/UserContactEditForm.tsx
index 3f5204b5b30..05b21cb81f2 100644
--- a/web-client/src/views/UserContactEditForm.tsx
+++ b/web-client/src/views/UserContactEditForm.tsx
@@ -30,7 +30,7 @@ export const UserContactEditForm = connect(
         <Country
           bind={bind}
           type={type}
-          onBlur={() => validateUserContactSequence({ isSubmitting: false })}
+          onBlur={validateUserContactSequence}
           onChange={'updateFormValueSequence'}
           onChangeCountryType={'changeCountryTypeSequence'}
         />
@@ -38,14 +38,14 @@ export const UserContactEditForm = connect(
           <Address
             bind={bind}
             type={type}
-            onBlur={() => validateUserContactSequence({ isSubmitting: false })}
+            onBlur={validateUserContactSequence}
             onChange={'updateFormValueSequence'}
           />
         ) : (
           <InternationalAddress
             bind={bind}
             type={type}
-            onBlur={() => validateUserContactSequence({ isSubmitting: false })}
+            onBlur={validateUserContactSequence}
             onChange={'updateFormValueSequence'}
           />
         )}
@@ -64,7 +64,7 @@ export const UserContactEditForm = connect(
             name="contact.phone"
             type="text"
             value={form.contact.phone || ''}
-            onBlur={() => validateUserContactSequence({ isSubmitting: false })}
+            onBlur={() => validateUserContactSequence()}
             onChange={e => {
               updateFormValueSequence({
                 key: e.target.name,

From 8b3c5a57a3b9972afebc269d6d5fd4f3e5a57594 Mon Sep 17 00:00:00 2001
From: Christopher Bisom <cbisom@flexion.us>
Date: Thu, 19 Sep 2024 18:55:39 -0400
Subject: [PATCH 13/33] 10074-bug: fix other forms too

---
 .../sequences/uploadCorrespondenceDocumentSequence.ts       | 4 ++--
 .../sequences/uploadCourtIssuedDocumentSequence.ts          | 4 ++--
 .../validateUploadCorrespondenceDocumentSequence.ts         | 6 +++++-
 .../sequences/validateUploadCourtIssuedDocumentSequence.ts  | 6 +++++-
 web-client/src/views/ScanBatchPreviewer.tsx                 | 1 +
 5 files changed, 15 insertions(+), 6 deletions(-)

diff --git a/web-client/src/presenter/sequences/uploadCorrespondenceDocumentSequence.ts b/web-client/src/presenter/sequences/uploadCorrespondenceDocumentSequence.ts
index 5d15fbeea40..9ddb1fe0b76 100644
--- a/web-client/src/presenter/sequences/uploadCorrespondenceDocumentSequence.ts
+++ b/web-client/src/presenter/sequences/uploadCorrespondenceDocumentSequence.ts
@@ -3,7 +3,6 @@ import { clearAlertsAction } from '../actions/clearAlertsAction';
 import { getUploadCorrespondenceDocumentAlertSuccessAction } from '../actions/CorrespondenceDocument/getUploadCorrespondenceDocumentAlertSuccessAction';
 import { navigateToCaseDetailAction } from '../actions/navigateToCaseDetailAction';
 import { openFileUploadErrorModal } from '../actions/openFileUploadErrorModal';
-import { setAlertErrorAction } from '../actions/setAlertErrorAction';
 import { setAlertSuccessAction } from '../actions/setAlertSuccessAction';
 import { setCaseAction } from '../actions/setCaseAction';
 import { setCaseDetailPageTabActionGenerator } from '../actions/setCaseDetailPageTabActionGenerator';
@@ -11,6 +10,7 @@ import { setCaseDetailPageTabFrozenAction } from '../actions/CaseDetail/setCaseD
 import { setCorrespondenceIdToDisplayAction } from '../actions/CorrespondenceDocument/setCorrespondenceIdToDisplayAction';
 import { setDocumentTitleFromFormAction } from '../actions/CorrespondenceDocument/setDocumentTitleFromFormAction';
 import { setSaveAlertsForNavigationAction } from '../actions/setSaveAlertsForNavigationAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
 import { setShowModalFactoryAction } from '../actions/setShowModalFactoryAction';
 import { setValidationAlertErrorsAction } from '../actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
@@ -30,8 +30,8 @@ export const uploadCorrespondenceDocumentSequence = [
       validateUploadCorrespondenceDocumentAction,
       {
         error: [
-          setAlertErrorAction,
           setValidationErrorsAction,
+          setScrollToErrorNotificationAction,
           setValidationAlertErrorsAction,
         ],
         success: showProgressSequenceDecorator([
diff --git a/web-client/src/presenter/sequences/uploadCourtIssuedDocumentSequence.ts b/web-client/src/presenter/sequences/uploadCourtIssuedDocumentSequence.ts
index 099eaf0ee46..5fd766188ff 100644
--- a/web-client/src/presenter/sequences/uploadCourtIssuedDocumentSequence.ts
+++ b/web-client/src/presenter/sequences/uploadCourtIssuedDocumentSequence.ts
@@ -2,7 +2,6 @@ import { clearAlertsAction } from '../actions/clearAlertsAction';
 import { getUploadCourtIssuedDocumentAlertSuccessAction } from '../actions/UploadCourtIssuedDocument/getUploadCourtIssuedDocumentAlertSuccessAction';
 import { navigateToCaseDetailAction } from '../actions/navigateToCaseDetailAction';
 import { openFileUploadErrorModal } from '../actions/openFileUploadErrorModal';
-import { setAlertErrorAction } from '../actions/setAlertErrorAction';
 import { setAlertSuccessAction } from '../actions/setAlertSuccessAction';
 import { setCaseAction } from '../actions/setCaseAction';
 import { setCaseDetailPageTabActionGenerator } from '../actions/setCaseDetailPageTabActionGenerator';
@@ -11,6 +10,7 @@ import { setDefaultDraftDocumentIdAction } from '../actions/setDefaultDraftDocum
 import { setDocumentTitleFromFreeTextAction } from '../actions/UploadCourtIssuedDocument/setDocumentTitleFromFreeTextAction';
 import { setIsPrimaryTabAction } from '../actions/setIsPrimaryTabAction';
 import { setSaveAlertsForNavigationAction } from '../actions/setSaveAlertsForNavigationAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
 import { setValidationAlertErrorsAction } from '../actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
 import { showProgressSequenceDecorator } from '../utilities/showProgressSequenceDecorator';
@@ -25,8 +25,8 @@ export const uploadCourtIssuedDocumentSequence = [
   validateUploadCourtIssuedDocumentAction,
   {
     error: [
-      setAlertErrorAction,
       setValidationErrorsAction,
+      setScrollToErrorNotificationAction,
       setValidationAlertErrorsAction,
     ],
     success: showProgressSequenceDecorator([
diff --git a/web-client/src/presenter/sequences/validateUploadCorrespondenceDocumentSequence.ts b/web-client/src/presenter/sequences/validateUploadCorrespondenceDocumentSequence.ts
index 8d96e4f562d..7df2783e74b 100644
--- a/web-client/src/presenter/sequences/validateUploadCorrespondenceDocumentSequence.ts
+++ b/web-client/src/presenter/sequences/validateUploadCorrespondenceDocumentSequence.ts
@@ -1,4 +1,5 @@
 import { clearAlertsAction } from '../actions/clearAlertsAction';
+import { setValidationAlertErrorsAction } from '@web-client/presenter/actions/setValidationAlertErrorsAction';
 import { setValidationErrorsByFlagAction } from '../actions/WorkItem/setValidationErrorsByFlagAction';
 import { shouldValidateAction } from '../actions/shouldValidateAction';
 import { validateUploadCorrespondenceDocumentAction } from '../actions/CorrespondenceDocument/validateUploadCorrespondenceDocumentAction';
@@ -10,7 +11,10 @@ export const validateUploadCorrespondenceDocumentSequence = [
     validate: [
       validateUploadCorrespondenceDocumentAction,
       {
-        error: [setValidationErrorsByFlagAction],
+        error: [
+          setValidationErrorsByFlagAction,
+          setValidationAlertErrorsAction,
+        ],
         success: [clearAlertsAction],
       },
     ],
diff --git a/web-client/src/presenter/sequences/validateUploadCourtIssuedDocumentSequence.ts b/web-client/src/presenter/sequences/validateUploadCourtIssuedDocumentSequence.ts
index 6aaf67bdd5a..bd681dd7180 100644
--- a/web-client/src/presenter/sequences/validateUploadCourtIssuedDocumentSequence.ts
+++ b/web-client/src/presenter/sequences/validateUploadCourtIssuedDocumentSequence.ts
@@ -1,4 +1,5 @@
 import { clearAlertsAction } from '../actions/clearAlertsAction';
+import { setValidationAlertErrorsAction } from '@web-client/presenter/actions/setValidationAlertErrorsAction';
 import { setValidationErrorsByFlagAction } from '../actions/WorkItem/setValidationErrorsByFlagAction';
 import { shouldValidateAction } from '../actions/shouldValidateAction';
 import { validateUploadCourtIssuedDocumentAction } from '../actions/UploadCourtIssuedDocument/validateUploadCourtIssuedDocumentAction';
@@ -10,7 +11,10 @@ export const validateUploadCourtIssuedDocumentSequence = [
     validate: [
       validateUploadCourtIssuedDocumentAction,
       {
-        error: [setValidationErrorsByFlagAction],
+        error: [
+          setValidationErrorsByFlagAction,
+          setValidationAlertErrorsAction,
+        ],
         success: [clearAlertsAction],
       },
     ],
diff --git a/web-client/src/views/ScanBatchPreviewer.tsx b/web-client/src/views/ScanBatchPreviewer.tsx
index c6db9b375c9..674e87d5267 100644
--- a/web-client/src/views/ScanBatchPreviewer.tsx
+++ b/web-client/src/views/ScanBatchPreviewer.tsx
@@ -282,6 +282,7 @@ export const ScanBatchPreviewer = connect(
             <ConfirmDeletePDFModal
               confirmSequence={() => {
                 removeScannedPdfSequence();
+                if (validateSequence) validateSequence();
               }}
               confirmText="Yes, Delete"
               modalContent="This action cannot be undone."

From 02014eb44c8ccb4c6c2ecf1910aec6eb6f0e0fa6 Mon Sep 17 00:00:00 2001
From: Christopher Bisom <cbisom@flexion.us>
Date: Thu, 19 Sep 2024 19:00:35 -0400
Subject: [PATCH 14/33] 10074-bug: fix document selection form

---
 .../presenter/sequences/completeDocumentSelectSequence.ts  | 7 ++++++-
 .../sequences/reviewExternalDocumentInformationSequence.ts | 2 ++
 2 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/web-client/src/presenter/sequences/completeDocumentSelectSequence.ts b/web-client/src/presenter/sequences/completeDocumentSelectSequence.ts
index 35119c062bd..735f071d264 100644
--- a/web-client/src/presenter/sequences/completeDocumentSelectSequence.ts
+++ b/web-client/src/presenter/sequences/completeDocumentSelectSequence.ts
@@ -6,6 +6,7 @@ import { navigateToFileADocumentAction } from '../actions/FileDocument/navigateT
 import { refreshExternalDocumentTitleFromEventCodeAction } from '../actions/FileDocument/refreshExternalDocumentTitleFromEventCodeAction';
 import { setDefaultFileDocumentFormValuesAction } from '../actions/FileDocument/setDefaultFileDocumentFormValuesAction';
 import { setDocketNumberPropAction } from '../actions/FileDocument/setDocketNumberPropAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
 import { setValidationAlertErrorsAction } from '../actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
 import { startShowValidationAction } from '../actions/startShowValidationAction';
@@ -19,7 +20,11 @@ export const completeDocumentSelectSequence = [
   generateTitleAction,
   validateSelectDocumentTypeAction,
   {
-    error: [setValidationErrorsAction, setValidationAlertErrorsAction],
+    error: [
+      setValidationErrorsAction,
+      setScrollToErrorNotificationAction,
+      setValidationAlertErrorsAction,
+    ],
     success: [
       clearAlertsAction,
       stopShowValidationAction,
diff --git a/web-client/src/presenter/sequences/reviewExternalDocumentInformationSequence.ts b/web-client/src/presenter/sequences/reviewExternalDocumentInformationSequence.ts
index 91ab8a45bf5..28647077758 100644
--- a/web-client/src/presenter/sequences/reviewExternalDocumentInformationSequence.ts
+++ b/web-client/src/presenter/sequences/reviewExternalDocumentInformationSequence.ts
@@ -6,6 +6,7 @@ import { navigateToReviewFileADocumentAction } from '../actions/FileDocument/nav
 import { setAlertErrorAction } from '../actions/setAlertErrorAction';
 import { setFilersFromFilersMapAction } from '../actions/setFilersFromFilersMapAction';
 import { setPdfPreviewUrlAction } from '@web-client/presenter/actions/CourtIssuedOrder/setPdfPreviewUrlAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
 import { setSupportingDocumentScenarioAction } from '../actions/FileDocument/setSupportingDocumentScenarioAction';
 import { setValidationAlertErrorsAction } from '../actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
@@ -24,6 +25,7 @@ export const reviewExternalDocumentInformationSequence =
       error: [
         setAlertErrorAction,
         setValidationErrorsAction,
+        setScrollToErrorNotificationAction,
         setValidationAlertErrorsAction,
       ],
       success: [

From c5b0ef883442362da18fc3fba84d8c44bfea3df8 Mon Sep 17 00:00:00 2001
From: Christopher Bisom <cbisom@flexion.us>
Date: Thu, 19 Sep 2024 19:05:53 -0400
Subject: [PATCH 15/33] 10074-bug: fix a couple more forms

---
 .../StatusReportOrder/submitStatusReportOrderSequence.ts   | 7 ++++++-
 .../sequences/editUploadCourtIssuedDocumentSequence.ts     | 2 ++
 2 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/web-client/src/presenter/sequences/StatusReportOrder/submitStatusReportOrderSequence.ts b/web-client/src/presenter/sequences/StatusReportOrder/submitStatusReportOrderSequence.ts
index 7943c84dbd9..e93067de6ca 100644
--- a/web-client/src/presenter/sequences/StatusReportOrder/submitStatusReportOrderSequence.ts
+++ b/web-client/src/presenter/sequences/StatusReportOrder/submitStatusReportOrderSequence.ts
@@ -2,6 +2,7 @@ import { clearAlertsAction } from '@web-client/presenter/actions/clearAlertsActi
 import { navigateToPathAction } from '@web-client/presenter/actions/navigateToPathAction';
 import { prepareStatusReportOrderAction } from '../../actions/StatusReportOrder/prepareStatusReportOrderAction';
 import { setAlertErrorAction } from '@web-client/presenter/actions/setAlertErrorAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
 import { setValidationAlertErrorsAction } from '@web-client/presenter/actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../../actions/setValidationErrorsAction';
 import { showProgressSequenceDecorator } from '../../utilities/showProgressSequenceDecorator';
@@ -13,7 +14,11 @@ export const submitStatusReportOrderSequence = [
     clearAlertsAction,
     validateStatusReportOrderFormAction,
     {
-      error: [setValidationErrorsAction, setValidationAlertErrorsAction],
+      error: [
+        setValidationErrorsAction,
+        setScrollToErrorNotificationAction,
+        setValidationAlertErrorsAction,
+      ],
       success: [
         prepareStatusReportOrderAction,
         submitCourtIssuedOrderSequence,
diff --git a/web-client/src/presenter/sequences/editUploadCourtIssuedDocumentSequence.ts b/web-client/src/presenter/sequences/editUploadCourtIssuedDocumentSequence.ts
index 9be608ed1c4..5a74e89e6ca 100644
--- a/web-client/src/presenter/sequences/editUploadCourtIssuedDocumentSequence.ts
+++ b/web-client/src/presenter/sequences/editUploadCourtIssuedDocumentSequence.ts
@@ -16,6 +16,7 @@ import { setDocumentTitleFromFreeTextAction } from '../actions/UploadCourtIssued
 import { setIsPrimaryTabAction } from '../actions/setIsPrimaryTabAction';
 import { setPrimaryDocumentFileIdPropAction } from '../actions/EditUploadCourtIssuedDocument/setPrimaryDocumentFileIdPropAction';
 import { setSaveAlertsForNavigationAction } from '../actions/setSaveAlertsForNavigationAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
 import { setValidationAlertErrorsAction } from '../actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
 import { setupUploadMetadataAction } from '../actions/UploadCourtIssuedDocument/setupUploadMetadataAction';
@@ -53,6 +54,7 @@ export const editUploadCourtIssuedDocumentSequence = [
     error: [
       setAlertErrorAction,
       setValidationErrorsAction,
+      setScrollToErrorNotificationAction,
       setValidationAlertErrorsAction,
     ],
     success: showProgressSequenceDecorator([

From 6f7b9db8644127e3db7ffd6f901f3fedd9209a37 Mon Sep 17 00:00:00 2001
From: Christopher Bisom <cbisom@flexion.us>
Date: Fri, 20 Sep 2024 08:21:14 -0400
Subject: [PATCH 16/33] 10074-bug: add live validation to status report order
 form

---
 .../journey/petitionsClerkUpdatesCaseDetail.ts   |  2 +-
 .../actions/setValidationAlertErrorsAction.ts    |  2 +-
 web-client/src/presenter/presenter.ts            |  2 ++
 .../validateStatusReportOrderSequence.ts         | 16 ++++++++++++++++
 web-client/src/views/ErrorNotification.tsx       |  4 ++--
 web-client/src/views/StatusReportOrder.tsx       |  6 ++++++
 6 files changed, 28 insertions(+), 4 deletions(-)
 create mode 100644 web-client/src/presenter/sequences/StatusReportOrder/validateStatusReportOrderSequence.ts

diff --git a/web-client/integration-tests/journey/petitionsClerkUpdatesCaseDetail.ts b/web-client/integration-tests/journey/petitionsClerkUpdatesCaseDetail.ts
index 2b630a35a9d..f574dc4f233 100644
--- a/web-client/integration-tests/journey/petitionsClerkUpdatesCaseDetail.ts
+++ b/web-client/integration-tests/journey/petitionsClerkUpdatesCaseDetail.ts
@@ -143,7 +143,7 @@ export const petitionsClerkUpdatesCaseDetail = cerebralTest => {
     });
     expect(cerebralTest.getState('alertError')).toEqual({
       messages: ['Select a case type', 'Select a case procedure'],
-      preventAutoScroll: false,
+      scrollToErrorNotification: false,
       title: 'Please correct the following errors on the page:',
     });
 
diff --git a/web-client/src/presenter/actions/setValidationAlertErrorsAction.ts b/web-client/src/presenter/actions/setValidationAlertErrorsAction.ts
index 7af5683692a..6bfe5a31d8b 100644
--- a/web-client/src/presenter/actions/setValidationAlertErrorsAction.ts
+++ b/web-client/src/presenter/actions/setValidationAlertErrorsAction.ts
@@ -86,7 +86,7 @@ export const setValidationAlertErrorsAction = ({
           }
         }),
     ),
-    preventAutoScroll: !props.scrollToErrorNotification,
+    scrollToErrorNotification: props.scrollToErrorNotification || false,
     title: 'Please correct the following errors on the page:',
   };
   store.set(state.alertError, alertError);
diff --git a/web-client/src/presenter/presenter.ts b/web-client/src/presenter/presenter.ts
index b3c827ac155..14e94179d2a 100644
--- a/web-client/src/presenter/presenter.ts
+++ b/web-client/src/presenter/presenter.ts
@@ -604,6 +604,7 @@ import { validateRemoveFromTrialSessionSequence } from './sequences/validateRemo
 import { validateSelectDocumentTypeSequence } from './sequences/validateSelectDocumentTypeSequence';
 import { validateSetForHearingSequence } from './sequences/validateSetForHearingSequence';
 import { validateStampSequence } from './sequences/validateStampSequence';
+import { validateStatusReportOrderSequence } from '@web-client/presenter/sequences/StatusReportOrder/validateStatusReportOrderSequence';
 import { validateTrialSessionHearingNoteSequence } from './sequences/validateTrialSessionHearingNoteSequence';
 import { validateTrialSessionNoteSequence } from './sequences/validateTrialSessionNoteSequence';
 import { validateTrialSessionPlanningSequence } from './sequences/validateTrialSessionPlanningSequence';
@@ -1594,6 +1595,7 @@ export const presenterSequences = {
   validateSetForHearingSequence:
     validateSetForHearingSequence as unknown as Function,
   validateStampSequence: validateStampSequence as unknown as Function,
+  validateStatusReportOrderSequence,
   validateTrialSessionHearingNoteSequence:
     validateTrialSessionHearingNoteSequence as unknown as Function,
   validateTrialSessionNoteSequence:
diff --git a/web-client/src/presenter/sequences/StatusReportOrder/validateStatusReportOrderSequence.ts b/web-client/src/presenter/sequences/StatusReportOrder/validateStatusReportOrderSequence.ts
new file mode 100644
index 00000000000..362313cdb7d
--- /dev/null
+++ b/web-client/src/presenter/sequences/StatusReportOrder/validateStatusReportOrderSequence.ts
@@ -0,0 +1,16 @@
+import { clearAlertsAction } from '@web-client/presenter/actions/clearAlertsAction';
+import { setValidationAlertErrorsAction } from '@web-client/presenter/actions/setValidationAlertErrorsAction';
+import { setValidationErrorsAction } from '../../actions/setValidationErrorsAction';
+import { showProgressSequenceDecorator } from '../../utilities/showProgressSequenceDecorator';
+import { validateStatusReportOrderFormAction } from '../../actions/StatusReportOrder/validateStatusReportOrderFormAction';
+
+export const validateStatusReportOrderSequence = [
+  showProgressSequenceDecorator([
+    clearAlertsAction,
+    validateStatusReportOrderFormAction,
+    {
+      error: [setValidationErrorsAction, setValidationAlertErrorsAction],
+      success: [clearAlertsAction],
+    },
+  ]),
+] as unknown as () => void;
diff --git a/web-client/src/views/ErrorNotification.tsx b/web-client/src/views/ErrorNotification.tsx
index 6e80076d0fa..ee9152ae300 100644
--- a/web-client/src/views/ErrorNotification.tsx
+++ b/web-client/src/views/ErrorNotification.tsx
@@ -15,7 +15,7 @@ export const ErrorNotification = connect(
     alertError?: {
       title?: string;
       message?: string;
-      preventAutoScroll?: boolean;
+      scrollToErrorNotification?: boolean;
     };
     alertHelper: {
       showErrorAlert?: boolean;
@@ -29,7 +29,7 @@ export const ErrorNotification = connect(
 
     useEffect(() => {
       const notification = notificationRef.current;
-      if (notification && !alertError?.preventAutoScroll) {
+      if (notification && alertError?.scrollToErrorNotification) {
         window.scrollTo(0, 0);
       }
     });
diff --git a/web-client/src/views/StatusReportOrder.tsx b/web-client/src/views/StatusReportOrder.tsx
index 3bb93a29223..7fc49e45ce1 100644
--- a/web-client/src/views/StatusReportOrder.tsx
+++ b/web-client/src/views/StatusReportOrder.tsx
@@ -24,6 +24,8 @@ export const StatusReportOrder = connect(
       sequences.statusReportOrderPdfPreviewSequence,
     submitStatusReportOrderSequence: sequences.submitStatusReportOrderSequence,
     updateFormValueSequence: sequences.updateStatusReportOrderFormValueSequence,
+    validateStatusReportOrderSequence:
+      sequences.validateStatusReportOrderSequence,
     validationErrors: state.validationErrors,
   },
   function StatusReportOrder({
@@ -36,6 +38,7 @@ export const StatusReportOrder = connect(
     statusReportOrderPdfPreviewSequence,
     submitStatusReportOrderSequence,
     updateFormValueSequence,
+    validateStatusReportOrderSequence,
     validationErrors,
   }) {
     return (
@@ -226,6 +229,7 @@ export const StatusReportOrder = connect(
                           toFormat: constants.DATE_FORMATS.YYYYMMDD,
                           value: e.target.value,
                         });
+                        validateStatusReportOrderSequence();
                       }}
                     />
                   </FormGroup>
@@ -294,6 +298,7 @@ export const StatusReportOrder = connect(
                             key: e.target.name,
                             value: e.target.value,
                           });
+                          validateStatusReportOrderSequence();
                         }}
                       />
                       <label
@@ -326,6 +331,7 @@ export const StatusReportOrder = connect(
                             key: e.target.name,
                             value: e.target.value,
                           });
+                          validateStatusReportOrderSequence();
                         }}
                       />
                       <label

From 9ba26822e48baf13764e970019c8125c5934b826 Mon Sep 17 00:00:00 2001
From: Christopher Bisom <cbisom@flexion.us>
Date: Fri, 20 Sep 2024 09:08:26 -0400
Subject: [PATCH 17/33] 10074-bug: fix EditDocketEntryMeta

---
 .../sequences/submitEditDocketEntryMetaSequence.ts          | 3 ++-
 .../src/presenter/sequences/validateDocumentSequence.ts     | 6 +++++-
 2 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/web-client/src/presenter/sequences/submitEditDocketEntryMetaSequence.ts b/web-client/src/presenter/sequences/submitEditDocketEntryMetaSequence.ts
index 0080660b224..a1709a8c266 100644
--- a/web-client/src/presenter/sequences/submitEditDocketEntryMetaSequence.ts
+++ b/web-client/src/presenter/sequences/submitEditDocketEntryMetaSequence.ts
@@ -14,6 +14,7 @@ import { setAlertErrorAction } from '../actions/setAlertErrorAction';
 import { setAlertSuccessAction } from '../actions/setAlertSuccessAction';
 import { setFilersFromFilersMapAction } from '../actions/setFilersFromFilersMapAction';
 import { setSaveAlertsForNavigationAction } from '../actions/setSaveAlertsForNavigationAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
 import { setValidationAlertErrorsAction } from '../actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
 import { setupUploadMetadataAction } from '../actions/UploadCourtIssuedDocument/setupUploadMetadataAction';
@@ -43,8 +44,8 @@ export const submitEditDocketEntryMetaSequence = [
   validateDocumentAction,
   {
     error: [
-      setAlertErrorAction,
       setValidationErrorsAction,
+      setScrollToErrorNotificationAction,
       setValidationAlertErrorsAction,
     ],
     success: showProgressSequenceDecorator([
diff --git a/web-client/src/presenter/sequences/validateDocumentSequence.ts b/web-client/src/presenter/sequences/validateDocumentSequence.ts
index 41520a02183..274ae88e4ff 100644
--- a/web-client/src/presenter/sequences/validateDocumentSequence.ts
+++ b/web-client/src/presenter/sequences/validateDocumentSequence.ts
@@ -1,5 +1,6 @@
 import { clearAlertsAction } from '../actions/clearAlertsAction';
 import { setFilersFromFilersMapAction } from '../actions/setFilersFromFilersMapAction';
+import { setValidationAlertErrorsAction } from '@web-client/presenter/actions/setValidationAlertErrorsAction';
 import { setValidationErrorsByFlagAction } from '../actions/WorkItem/setValidationErrorsByFlagAction';
 import { shouldValidateAction } from '../actions/shouldValidateAction';
 import { validateDocumentAction } from '../actions/EditDocketRecordEntry/validateDocumentAction';
@@ -12,7 +13,10 @@ export const validateDocumentSequence = [
       setFilersFromFilersMapAction,
       validateDocumentAction,
       {
-        error: [setValidationErrorsByFlagAction],
+        error: [
+          setValidationErrorsByFlagAction,
+          setValidationAlertErrorsAction,
+        ],
         success: [clearAlertsAction],
       },
     ],

From 7753a96eb99afea7480ae537bff279c259f9e6e1 Mon Sep 17 00:00:00 2001
From: Christopher Bisom <cbisom@flexion.us>
Date: Fri, 20 Sep 2024 12:51:50 -0400
Subject: [PATCH 18/33] 10074-bug: fix docektEntryQC

---
 .../src/presenter/sequences/completeDocketEntryQCSequence.ts  | 4 ++--
 .../sequences/openCompleteAndSendMessageModalSequence.ts      | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/web-client/src/presenter/sequences/completeDocketEntryQCSequence.ts b/web-client/src/presenter/sequences/completeDocketEntryQCSequence.ts
index 7df5815529c..c7ba917a4b8 100644
--- a/web-client/src/presenter/sequences/completeDocketEntryQCSequence.ts
+++ b/web-client/src/presenter/sequences/completeDocketEntryQCSequence.ts
@@ -5,7 +5,6 @@ import { getCaseAction } from '../actions/getCaseAction';
 import { isWorkItemAlreadyCompletedAction } from '../actions/isWorkItemAlreadyCompletedAction';
 import { navigateToDocumentQCAction } from '../actions/navigateToDocumentQCAction';
 import { refreshExternalDocumentTitleFromEventCodeAction } from '../actions/FileDocument/refreshExternalDocumentTitleFromEventCodeAction';
-import { setAlertErrorAction } from '../actions/setAlertErrorAction';
 import { setAlertSuccessAction } from '../actions/setAlertSuccessAction';
 import { setCaseAction } from '../actions/setCaseAction';
 import { setFilersFromFilersMapAction } from '../actions/setFilersFromFilersMapAction';
@@ -13,6 +12,7 @@ import { setPaperServicePartiesAction } from '../actions/setPaperServicePartiesA
 import { setPdfPreviewUrlAction } from '../actions/CourtIssuedOrder/setPdfPreviewUrlAction';
 import { setPreviousDocumentDocketEntryAction } from '../actions/FileDocument/setPreviousDocumentDocketEntryAction';
 import { setSaveAlertsForNavigationAction } from '../actions/setSaveAlertsForNavigationAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
 import { setShowModalFactoryAction } from '../actions/setShowModalFactoryAction';
 import { setValidationAlertErrorsAction } from '../actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
@@ -33,8 +33,8 @@ export const completeDocketEntryQCSequence = [
       validateDocketEntryAction,
       {
         error: [
-          setAlertErrorAction,
           setValidationErrorsAction,
+          setScrollToErrorNotificationAction,
           setValidationAlertErrorsAction,
         ],
         success: showProgressSequenceDecorator([
diff --git a/web-client/src/presenter/sequences/openCompleteAndSendMessageModalSequence.ts b/web-client/src/presenter/sequences/openCompleteAndSendMessageModalSequence.ts
index 72b6b6da4af..7e125543e4b 100644
--- a/web-client/src/presenter/sequences/openCompleteAndSendMessageModalSequence.ts
+++ b/web-client/src/presenter/sequences/openCompleteAndSendMessageModalSequence.ts
@@ -3,10 +3,10 @@ import { generateTitleAction } from '../actions/FileDocument/generateTitleAction
 import { getCaseAction } from '../actions/getCaseAction';
 import { isWorkItemAlreadyCompletedAction } from '../actions/isWorkItemAlreadyCompletedAction';
 import { refreshExternalDocumentTitleFromEventCodeAction } from '../actions/FileDocument/refreshExternalDocumentTitleFromEventCodeAction';
-import { setAlertErrorAction } from '../actions/setAlertErrorAction';
 import { setCaseAction } from '../actions/setCaseAction';
 import { setFilersFromFilersMapAction } from '../actions/setFilersFromFilersMapAction';
 import { setPreviousDocumentDocketEntryAction } from '../actions/FileDocument/setPreviousDocumentDocketEntryAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
 import { setShowModalFactoryAction } from '../actions/setShowModalFactoryAction';
 import { setValidationAlertErrorsAction } from '../actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
@@ -23,8 +23,8 @@ export const openCompleteAndSendMessageModalSequence = [
       validateDocketEntryAction,
       {
         error: [
-          setAlertErrorAction,
           setValidationErrorsAction,
+          setScrollToErrorNotificationAction,
           setValidationAlertErrorsAction,
         ],
         success: [

From eb57afe2315626b8c82c04d31b157a52c8c7f997 Mon Sep 17 00:00:00 2001
From: Christopher Bisom <cbisom@flexion.us>
Date: Fri, 20 Sep 2024 15:09:25 -0400
Subject: [PATCH 19/33] 10074-bug: fix more forms

---
 shared/src/business/entities/cases/Case.ts                   | 5 +++--
 .../src/presenter/actions/validateCaseDetailsAction.ts       | 2 ++
 .../src/presenter/sequences/validateCaseDetailsSequence.ts   | 3 ++-
 web-client/src/views/CaseDetail/PetitionPaymentForm.tsx      | 2 +-
 web-client/src/views/StartCase/ContactPrimary.tsx            | 3 ++-
 web-client/src/views/StartCaseInternal/Parties.tsx           | 3 ++-
 6 files changed, 12 insertions(+), 6 deletions(-)

diff --git a/shared/src/business/entities/cases/Case.ts b/shared/src/business/entities/cases/Case.ts
index 635f4d9e7f0..b881ecf4b6a 100644
--- a/shared/src/business/entities/cases/Case.ts
+++ b/shared/src/business/entities/cases/Case.ts
@@ -590,7 +590,7 @@ export class Case extends JoiValidationEntity {
       'petitionPaymentStatus',
       {
         is: PAYMENT_STATUS.PAID,
-        otherwise: joi.optional().allow(null),
+        otherwise: joi.allow(null).optional(),
         then: JoiValidationConstants.ISO_DATE.max('now').required(),
       },
     )
@@ -599,7 +599,7 @@ export class Case extends JoiValidationEntity {
     petitionPaymentMethod: JoiValidationConstants.STRING.max(50)
       .when('petitionPaymentStatus', {
         is: PAYMENT_STATUS.PAID,
-        otherwise: joi.optional().allow(null),
+        otherwise: joi.allow(null).optional(),
         then: joi.required(),
       })
       .description('How the petitioner paid the case fee.')
@@ -729,6 +729,7 @@ export class Case extends JoiValidationEntity {
   }
 
   assignFieldsForAllUsers({ rawCase }) {
+    console.log(rawCase);
     this.caseCaption = rawCase.caseCaption;
     this.caseType = rawCase.caseType;
     this.closedDate = rawCase.closedDate;
diff --git a/web-client/src/presenter/actions/validateCaseDetailsAction.ts b/web-client/src/presenter/actions/validateCaseDetailsAction.ts
index 65cf0ad457b..8895fb56edb 100644
--- a/web-client/src/presenter/actions/validateCaseDetailsAction.ts
+++ b/web-client/src/presenter/actions/validateCaseDetailsAction.ts
@@ -18,6 +18,8 @@ export const validateCaseDetailsAction = async ({
         caseDetail: {
           ...caseDetail,
           ...form,
+          petitionPaymentDate: form.petitionPaymentDate || null,
+          petitionPaymentMethod: form.petitionPaymentMethod || null,
           preferredTrialCity: form.preferredTrialCity
             ? form.preferredTrialCity
             : null,
diff --git a/web-client/src/presenter/sequences/validateCaseDetailsSequence.ts b/web-client/src/presenter/sequences/validateCaseDetailsSequence.ts
index 68eae5d9b2f..5b87e10cf0f 100644
--- a/web-client/src/presenter/sequences/validateCaseDetailsSequence.ts
+++ b/web-client/src/presenter/sequences/validateCaseDetailsSequence.ts
@@ -1,4 +1,5 @@
 import { clearAlertsAction } from '../actions/clearAlertsAction';
+import { setValidationAlertErrorsAction } from '@web-client/presenter/actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
 import { shouldValidateAction } from '../actions/shouldValidateAction';
 import { validateCaseDetailsAction } from '../actions/validateCaseDetailsAction';
@@ -10,7 +11,7 @@ export const validateCaseDetailsSequence = [
     validate: [
       validateCaseDetailsAction,
       {
-        error: [setValidationErrorsAction],
+        error: [setValidationErrorsAction, setValidationAlertErrorsAction],
         success: [clearAlertsAction],
       },
     ],
diff --git a/web-client/src/views/CaseDetail/PetitionPaymentForm.tsx b/web-client/src/views/CaseDetail/PetitionPaymentForm.tsx
index d913b5497c0..5fbba8ff003 100644
--- a/web-client/src/views/CaseDetail/PetitionPaymentForm.tsx
+++ b/web-client/src/views/CaseDetail/PetitionPaymentForm.tsx
@@ -142,10 +142,10 @@ const PetitionPaymentFormComponent: React.FC<ComponentProps & any> = ({
               onBlur={validateFormData}
               onChange={e => {
                 onUpdate({
-                  allowEmptyString: true,
                   key: e.target.name,
                   value: e.target.value,
                 });
+                validateFormData();
               }}
             />
           </FormGroup>
diff --git a/web-client/src/views/StartCase/ContactPrimary.tsx b/web-client/src/views/StartCase/ContactPrimary.tsx
index 063e1b90bc2..0f28fa6a5c2 100644
--- a/web-client/src/views/StartCase/ContactPrimary.tsx
+++ b/web-client/src/views/StartCase/ContactPrimary.tsx
@@ -13,7 +13,7 @@ import React from 'react';
 const props = cerebralProps as unknown as {
   contactsHelper: string;
   bind: string;
-  onBlur: (args: Record<string, any>) => void;
+  onBlur: () => void;
   onChange: string;
   parentView: string;
 };
@@ -98,6 +98,7 @@ export const ContactPrimary = connect(
           name="contactPrimary.title"
           type="text"
           value={data.contactPrimary.title || ''}
+          onBlur={onBlur}
           onChange={e => {
             onChangeSequence({
               key: e.target.name,
diff --git a/web-client/src/views/StartCaseInternal/Parties.tsx b/web-client/src/views/StartCaseInternal/Parties.tsx
index 5e36e2f1df4..e0ee684fa19 100644
--- a/web-client/src/views/StartCaseInternal/Parties.tsx
+++ b/web-client/src/views/StartCaseInternal/Parties.tsx
@@ -68,6 +68,7 @@ export const Parties = connect(
                   key: e.target.name,
                   value: e.target.checked,
                 });
+                validatePetitionFromPaperSequence();
               }}
             />
             <label
@@ -92,7 +93,7 @@ export const Parties = connect(
                 startCaseInternalHelper.showSecondaryContact
               }
               useSameAsPrimary={true}
-              onBlur={validatePetitionFromPaperSequence}
+              onBlur={() => validatePetitionFromPaperSequence()}
               onChange="updateFormValueAndCaseCaptionSequence"
             />
           </div>

From 0aba290894a0088beea35165e630945b16fc0392 Mon Sep 17 00:00:00 2001
From: Christopher Bisom <cbisom@flexion.us>
Date: Fri, 20 Sep 2024 15:25:23 -0400
Subject: [PATCH 20/33] 10074-bug: remove console.log

---
 shared/src/business/entities/cases/Case.ts | 1 -
 1 file changed, 1 deletion(-)

diff --git a/shared/src/business/entities/cases/Case.ts b/shared/src/business/entities/cases/Case.ts
index b881ecf4b6a..128d485146f 100644
--- a/shared/src/business/entities/cases/Case.ts
+++ b/shared/src/business/entities/cases/Case.ts
@@ -729,7 +729,6 @@ export class Case extends JoiValidationEntity {
   }
 
   assignFieldsForAllUsers({ rawCase }) {
-    console.log(rawCase);
     this.caseCaption = rawCase.caseCaption;
     this.caseType = rawCase.caseType;
     this.closedDate = rawCase.closedDate;

From 3f08d2322b773c961e6f22043ee59b34760d13f7 Mon Sep 17 00:00:00 2001
From: Christopher Bisom <cbisom@flexion.us>
Date: Fri, 20 Sep 2024 16:41:41 -0400
Subject: [PATCH 21/33] 10074-bug: update pretty much every submit sequence

---
 .../presenter/actions/validatePetitionerAction.ts  |  5 +----
 .../submitJudgeActivityReportSequence.ts           |  4 ++--
 .../submitJudgeActivityStatisticsReportSequence.ts |  4 ++--
 .../Login/submitChangePasswordSequence.ts          |  8 +++++++-
 .../editCorrespondenceDocumentSequence.ts          |  4 ++--
 .../editUploadCourtIssuedDocumentSequence.ts       |  2 --
 ...veCourtIssuedDocumentFromDocketEntrySequence.ts |  4 ++--
 .../sequences/filePetitionCompleteStep6Sequence.ts |  3 ++-
 ...nitiateCourtIssuedFilingServiceModalSequence.ts |  7 ++++++-
 .../reviewCaseAssociationRequestSequence.ts        |  4 ++--
 .../reviewExternalDocumentInformationSequence.ts   |  2 --
 .../saveCourtIssuedDocketEntrySequence.ts          |  4 ++--
 .../sequences/saveSavedCaseForLaterSequence.ts     |  4 ++--
 .../selectDateRangeFromCalendarSequence.ts         |  7 +------
 .../setJudgeActivityReportFiltersSequence.ts       |  7 +------
 .../submitAddDeficiencyStatisticsSequence.ts       |  3 ++-
 .../sequences/submitAddPetitionerSequence.ts       |  7 ++++++-
 .../submitAddPractitionerDocumentSequence.ts       |  4 ++--
 .../sequences/submitAddPractitionerSequence.ts     |  7 ++++++-
 .../submitChangeLoginAndServiceEmailSequence.ts    |  3 ++-
 .../submitCourtIssuedDocketEntrySequence.ts        |  4 ++--
 .../sequences/submitCourtIssuedOrderSequence.ts    |  7 ++++++-
 .../sequences/submitEditContactSequence.ts         |  8 +++++++-
 .../submitEditDeficiencyStatisticSequence.ts       |  3 ++-
 .../submitEditPetitionerCounselSequence.ts         |  8 +++++++-
 .../sequences/submitEditPetitionerSequence.ts      |  8 +++++++-
 .../submitEditPractitionerDocumentSequence.ts      |  4 ++--
 .../submitEditRespondentCounselSequence.ts         |  8 +++++++-
 .../sequences/submitStampMotionSequence.ts         |  4 ++--
 .../sequences/submitTrialSessionSequence.ts        |  1 -
 .../submitUpdatePractitionerUserSequence.ts        |  3 ++-
 .../submitUpdateUserContactInformationSequence.ts  |  7 ++++++-
 .../updateDateRangeForDeadlinesSequence.ts         |  2 --
 .../sequences/updateTrialSessionSequence.ts        |  3 ++-
 .../validateAddDeficiencyStatisticsSequence.ts     |  7 +------
 .../sequences/validatePetitionerSequence.ts        | 11 ++++++-----
 .../presenter/sequences/validateStampSequence.ts   |  7 +------
 .../sequences/validateUserContactSequence.ts       |  3 ++-
 web-client/src/views/ContactEdit.tsx               |  3 ++-
 .../views/EditPetitionerInformationInternal.tsx    | 14 ++++----------
 40 files changed, 118 insertions(+), 90 deletions(-)

diff --git a/web-client/src/presenter/actions/validatePetitionerAction.ts b/web-client/src/presenter/actions/validatePetitionerAction.ts
index 568b1f53b3b..d413421eb32 100644
--- a/web-client/src/presenter/actions/validatePetitionerAction.ts
+++ b/web-client/src/presenter/actions/validatePetitionerAction.ts
@@ -14,7 +14,6 @@ export const validatePetitionerAction = ({
   applicationContext,
   get,
   path,
-  store,
 }: ActionProps) => {
   const { contact } = get(state.form);
   const caseDetail = get(state.caseDetail);
@@ -26,11 +25,9 @@ export const validatePetitionerAction = ({
       existingPetitioners: caseDetail.petitioners,
     });
 
-  store.set(state.validationErrors.contact, errors);
-
   if (isEmpty(errors)) {
     return path.success();
   } else {
-    return path.error({ errors });
+    return path.error({ errors: { contact: errors } });
   }
 };
diff --git a/web-client/src/presenter/sequences/JudgeActivityReport/submitJudgeActivityReportSequence.ts b/web-client/src/presenter/sequences/JudgeActivityReport/submitJudgeActivityReportSequence.ts
index 2bf764ac720..99b41e26f34 100644
--- a/web-client/src/presenter/sequences/JudgeActivityReport/submitJudgeActivityReportSequence.ts
+++ b/web-client/src/presenter/sequences/JudgeActivityReport/submitJudgeActivityReportSequence.ts
@@ -6,11 +6,11 @@ import { getPendingMotionDocketEntriesAction } from '@web-client/presenter/actio
 import { getSubmittedAndCavCasesByJudgeAction } from '@web-client/presenter/actions/JudgeActivityReport/getSubmittedAndCavCasesByJudgeAction';
 import { parallel } from 'cerebral';
 import { resetHasUserSubmittedFormAction } from '@web-client/presenter/actions/JudgeActivityReport/resetHasUserSubmittedFormAction';
-import { setAlertErrorAction } from '../../actions/setAlertErrorAction';
 import { setCavAndSubmittedCasesAction } from '@web-client/presenter/actions/JudgeActivityReport/setCavAndSubmittedCasesAction';
 import { setDefaultSubmittedAndCavSortOrderAction } from '@web-client/presenter/actions/JudgeActivityReport/setDefaultSubmittedAndCavSortOrderAction';
 import { setJudgeActivityReportFiltersAction } from '@web-client/presenter/actions/JudgeActivityReport/setJudgeActivityReportFiltersAction';
 import { setPendingMotionDocketEntriesForCurrentJudgeAction } from '@web-client/presenter/actions/PendingMotion/setPendingMotionDocketEntriesForCurrentJudgeAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
 import { setValidationAlertErrorsAction } from '../../actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../../actions/setValidationErrorsAction';
 import { showProgressSequenceDecorator } from '@web-client/presenter/utilities/showProgressSequenceDecorator';
@@ -27,8 +27,8 @@ export const submitJudgeActivityReportSequence = showProgressSequenceDecorator([
   validateJudgeActivityReportSearchAction,
   {
     error: [
-      setAlertErrorAction,
       setValidationErrorsAction,
+      setScrollToErrorNotificationAction,
       setValidationAlertErrorsAction,
     ],
     success: [
diff --git a/web-client/src/presenter/sequences/JudgeActivityReport/submitJudgeActivityStatisticsReportSequence.ts b/web-client/src/presenter/sequences/JudgeActivityReport/submitJudgeActivityStatisticsReportSequence.ts
index 83a3a6e5215..8cf18e8f4c1 100644
--- a/web-client/src/presenter/sequences/JudgeActivityReport/submitJudgeActivityStatisticsReportSequence.ts
+++ b/web-client/src/presenter/sequences/JudgeActivityReport/submitJudgeActivityStatisticsReportSequence.ts
@@ -4,9 +4,9 @@ import { getCasesClosedByJudgeAction } from '../../actions/JudgeActivityReport/g
 import { getJudgeActivityReportCountsAction } from '../../actions/JudgeActivityReport/getJudgeActivityReportCountsAction';
 import { getTrialSessionsForJudgeActivityReportAction } from '../../actions/JudgeActivityReport/getTrialSessionsForJudgeActivityReportAction';
 import { parallel } from 'cerebral';
-import { setAlertErrorAction } from '../../actions/setAlertErrorAction';
 import { setHasUserSubmittedFormAction } from '@web-client/presenter/actions/JudgeActivityReport/setHasUserSubmittedFormAction';
 import { setJudgeActivityReportDataAction } from '@web-client/presenter/actions/JudgeActivityReport/setJudgeActivityReportDataAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
 import { setValidationAlertErrorsAction } from '../../actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../../actions/setValidationErrorsAction';
 import { showProgressSequenceDecorator } from '@web-client/presenter/utilities/showProgressSequenceDecorator';
@@ -21,8 +21,8 @@ export const submitJudgeActivityStatisticsReportSequence =
     validateJudgeActivityStatisticsReportSearchAction,
     {
       error: [
-        setAlertErrorAction,
         setValidationErrorsAction,
+        setScrollToErrorNotificationAction,
         setValidationAlertErrorsAction,
       ],
       success: [
diff --git a/web-client/src/presenter/sequences/Login/submitChangePasswordSequence.ts b/web-client/src/presenter/sequences/Login/submitChangePasswordSequence.ts
index 5fd1bcd7939..4e14b3c8eb5 100644
--- a/web-client/src/presenter/sequences/Login/submitChangePasswordSequence.ts
+++ b/web-client/src/presenter/sequences/Login/submitChangePasswordSequence.ts
@@ -5,10 +5,12 @@ import { navigateToLoginAction } from '@web-client/presenter/actions/Login/navig
 import { navigateToPathAction } from '@web-client/presenter/actions/navigateToPathAction';
 import { setAlertErrorAction } from '@web-client/presenter/actions/setAlertErrorAction';
 import { setSaveAlertsForNavigationAction } from '@web-client/presenter/actions/setSaveAlertsForNavigationAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
 import { setTokenAction } from '@web-client/presenter/actions/Login/setTokenAction';
 import { setUserAction } from '@web-client/presenter/actions/setUserAction';
 import { setUserPermissionsAction } from '@web-client/presenter/actions/setUserPermissionsAction';
 import { setValidationAlertErrorsAction } from '@web-client/presenter/actions/setValidationAlertErrorsAction';
+import { setValidationErrorsAction } from '@web-client/presenter/actions/setValidationErrorsAction';
 import { showProgressSequenceDecorator } from '../../utilities/showProgressSequenceDecorator';
 import { submitChangePasswordAction } from '@web-client/presenter/actions/Login/submitChangePasswordAction';
 import { validateChangePasswordFormAction } from '@web-client/presenter/actions/Login/validateChangePasswordFormAction';
@@ -18,7 +20,11 @@ export const submitChangePasswordSequence = [
     clearAlertsAction,
     validateChangePasswordFormAction,
     {
-      error: [setValidationAlertErrorsAction],
+      error: [
+        setValidationErrorsAction,
+        setScrollToErrorNotificationAction,
+        setValidationAlertErrorsAction,
+      ],
       success: [
         submitChangePasswordAction,
         {
diff --git a/web-client/src/presenter/sequences/editCorrespondenceDocumentSequence.ts b/web-client/src/presenter/sequences/editCorrespondenceDocumentSequence.ts
index 92fd4605a26..2ba2eaebd2a 100644
--- a/web-client/src/presenter/sequences/editCorrespondenceDocumentSequence.ts
+++ b/web-client/src/presenter/sequences/editCorrespondenceDocumentSequence.ts
@@ -5,7 +5,6 @@ import { getEditCorrespondenceDocumentAlertSuccessAction } from '../actions/Corr
 import { navigateToCaseDetailAction } from '../actions/navigateToCaseDetailAction';
 import { openFileUploadErrorModal } from '../actions/openFileUploadErrorModal';
 import { overwriteCorrespondenceFileAction } from '../actions/CourtIssuedOrder/overwriteCorrespondenceFileAction';
-import { setAlertErrorAction } from '../actions/setAlertErrorAction';
 import { setAlertSuccessAction } from '../actions/setAlertSuccessAction';
 import { setCaseAction } from '../actions/setCaseAction';
 import { setCaseDetailPageTabActionGenerator } from '../actions/setCaseDetailPageTabActionGenerator';
@@ -13,6 +12,7 @@ import { setCaseDetailPageTabFrozenAction } from '../actions/CaseDetail/setCaseD
 import { setDocumentTitleFromFormAction } from '../actions/CorrespondenceDocument/setDocumentTitleFromFormAction';
 import { setPrimaryCorrespondenceFileIdPropAction } from '../actions/CorrespondenceDocument/setPrimaryCorrespondenceFileIdPropAction';
 import { setSaveAlertsForNavigationAction } from '../actions/setSaveAlertsForNavigationAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
 import { setValidationAlertErrorsAction } from '../actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
 import { showProgressSequenceDecorator } from '../utilities/showProgressSequenceDecorator';
@@ -44,8 +44,8 @@ export const editCorrespondenceDocumentSequence = [
   validateUploadCorrespondenceDocumentAction,
   {
     error: [
-      setAlertErrorAction,
       setValidationErrorsAction,
+      setScrollToErrorNotificationAction,
       setValidationAlertErrorsAction,
     ],
     success: showProgressSequenceDecorator([
diff --git a/web-client/src/presenter/sequences/editUploadCourtIssuedDocumentSequence.ts b/web-client/src/presenter/sequences/editUploadCourtIssuedDocumentSequence.ts
index 5a74e89e6ca..3012f11163c 100644
--- a/web-client/src/presenter/sequences/editUploadCourtIssuedDocumentSequence.ts
+++ b/web-client/src/presenter/sequences/editUploadCourtIssuedDocumentSequence.ts
@@ -6,7 +6,6 @@ import { getUploadCourtIssuedDocumentAlertSuccessAction } from '../actions/Uploa
 import { navigateToCaseDetailAction } from '../actions/navigateToCaseDetailAction';
 import { openFileUploadErrorModal } from '../actions/openFileUploadErrorModal';
 import { overwriteOrderFileAction } from '../actions/CourtIssuedOrder/overwriteOrderFileAction';
-import { setAlertErrorAction } from '../actions/setAlertErrorAction';
 import { setAlertSuccessAction } from '../actions/setAlertSuccessAction';
 import { setCaseAction } from '../actions/setCaseAction';
 import { setCaseDetailPageTabActionGenerator } from '../actions/setCaseDetailPageTabActionGenerator';
@@ -52,7 +51,6 @@ export const editUploadCourtIssuedDocumentSequence = [
   validateUploadCourtIssuedDocumentAction,
   {
     error: [
-      setAlertErrorAction,
       setValidationErrorsAction,
       setScrollToErrorNotificationAction,
       setValidationAlertErrorsAction,
diff --git a/web-client/src/presenter/sequences/fileAndServeCourtIssuedDocumentFromDocketEntrySequence.ts b/web-client/src/presenter/sequences/fileAndServeCourtIssuedDocumentFromDocketEntrySequence.ts
index 2a5a1defc96..67f036e8bf8 100644
--- a/web-client/src/presenter/sequences/fileAndServeCourtIssuedDocumentFromDocketEntrySequence.ts
+++ b/web-client/src/presenter/sequences/fileAndServeCourtIssuedDocumentFromDocketEntrySequence.ts
@@ -5,7 +5,7 @@ import { computeJudgeNameWithTitleAction } from '../actions/computeJudgeNameWith
 import { fileAndServeCourtIssuedDocumentAction } from '../actions/CourtIssuedDocketEntry/fileAndServeCourtIssuedDocumentAction';
 import { generateCourtIssuedDocumentTitleAction } from '../actions/CourtIssuedDocketEntry/generateCourtIssuedDocumentTitleAction';
 import { getDocketNumbersForConsolidatedServiceAction } from '../actions/getDocketNumbersForConsolidatedServiceAction';
-import { setAlertErrorAction } from '../actions/setAlertErrorAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
 import { setValidationAlertErrorsAction } from '../actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
 import { setWaitingForResponseAction } from '../actions/setWaitingForResponseAction';
@@ -20,8 +20,8 @@ export const fileAndServeCourtIssuedDocumentFromDocketEntrySequence = [
   validateCourtIssuedDocketEntryAction,
   {
     error: [
-      setAlertErrorAction,
       setValidationErrorsAction,
+      setScrollToErrorNotificationAction,
       setValidationAlertErrorsAction,
       clearModalAction,
     ],
diff --git a/web-client/src/presenter/sequences/filePetitionCompleteStep6Sequence.ts b/web-client/src/presenter/sequences/filePetitionCompleteStep6Sequence.ts
index 8d1cb4f6826..f61206a099d 100644
--- a/web-client/src/presenter/sequences/filePetitionCompleteStep6Sequence.ts
+++ b/web-client/src/presenter/sequences/filePetitionCompleteStep6Sequence.ts
@@ -10,6 +10,7 @@ import { setAlertErrorAction } from '@web-client/presenter/actions/setAlertError
 import { setAlertSuccessAction } from '@web-client/presenter/actions/setAlertSuccessAction';
 import { setCaseAction } from '@web-client/presenter/actions/setCaseAction';
 import { setProgressForFileUploadAction } from '@web-client/presenter/actions/setProgressForFileUploadAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
 import { setValidationAlertErrorsAction } from '@web-client/presenter/actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '@web-client/presenter/actions/setValidationErrorsAction';
 import { setWaitingForResponseAction } from '@web-client/presenter/actions/setWaitingForResponseAction';
@@ -30,8 +31,8 @@ export const filePetitionCompleteStep6Sequence = debounceSequenceDecorator(
     {
       error: [
         unsetWaitingForResponseAction,
-        setAlertErrorAction,
         setValidationErrorsAction,
+        setScrollToErrorNotificationAction,
         setValidationAlertErrorsAction,
       ],
       success: [
diff --git a/web-client/src/presenter/sequences/openConfirmInitiateCourtIssuedFilingServiceModalSequence.ts b/web-client/src/presenter/sequences/openConfirmInitiateCourtIssuedFilingServiceModalSequence.ts
index 9828b722a31..0536fafb859 100644
--- a/web-client/src/presenter/sequences/openConfirmInitiateCourtIssuedFilingServiceModalSequence.ts
+++ b/web-client/src/presenter/sequences/openConfirmInitiateCourtIssuedFilingServiceModalSequence.ts
@@ -2,6 +2,7 @@ import { clearAlertsAction } from '../actions/clearAlertsAction';
 import { clearModalStateAction } from '../actions/clearModalStateAction';
 import { isDocketEntryMultiDocketableAction } from '../actions/CaseConsolidation/isDocketEntryMultiDocketableAction';
 import { setMultiDocketingCheckboxesAction } from '../actions/CaseConsolidation/setMultiDocketingCheckboxesAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
 import { setShowModalFactoryAction } from '../actions/setShowModalFactoryAction';
 import { setValidationAlertErrorsAction } from '../actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
@@ -13,7 +14,11 @@ export const openConfirmInitiateCourtIssuedFilingServiceModalSequence = [
   startShowValidationAction,
   validateCourtIssuedDocketEntryAction,
   {
-    error: [setValidationErrorsAction, setValidationAlertErrorsAction],
+    error: [
+      setValidationErrorsAction,
+      setScrollToErrorNotificationAction,
+      setValidationAlertErrorsAction,
+    ],
     success: [
       clearModalStateAction,
       isDocketEntryMultiDocketableAction,
diff --git a/web-client/src/presenter/sequences/reviewCaseAssociationRequestSequence.ts b/web-client/src/presenter/sequences/reviewCaseAssociationRequestSequence.ts
index 1fdfd04ce70..ed42115c708 100644
--- a/web-client/src/presenter/sequences/reviewCaseAssociationRequestSequence.ts
+++ b/web-client/src/presenter/sequences/reviewCaseAssociationRequestSequence.ts
@@ -3,9 +3,9 @@ import { generateCaseAssociationTitleAction } from '../actions/CaseAssociationRe
 import { generateEntryOfAppearancePdfAction } from '@web-client/presenter/actions/CaseAssociationRequest/generateEntryOfAppearancePdfAction';
 import { generateTitleForSupportingDocumentsAction } from '../actions/FileDocument/generateTitleForSupportingDocumentsAction';
 import { navigateToReviewCaseAssociationRequestAction } from '../actions/navigateToReviewCaseAssociationRequestAction';
-import { setAlertErrorAction } from '../actions/setAlertErrorAction';
 import { setFilersFromFilersMapAction } from '../actions/setFilersFromFilersMapAction';
 import { setPdfPreviewUrlAction } from '@web-client/presenter/actions/CourtIssuedOrder/setPdfPreviewUrlAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
 import { setSupportingDocumentScenarioAction } from '../actions/FileDocument/setSupportingDocumentScenarioAction';
 import { setValidationAlertErrorsAction } from '../actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
@@ -23,8 +23,8 @@ export const reviewCaseAssociationRequestSequence =
     validateCaseAssociationRequestAction,
     {
       error: [
-        setAlertErrorAction,
         setValidationErrorsAction,
+        setScrollToErrorNotificationAction,
         setValidationAlertErrorsAction,
       ],
       success: [
diff --git a/web-client/src/presenter/sequences/reviewExternalDocumentInformationSequence.ts b/web-client/src/presenter/sequences/reviewExternalDocumentInformationSequence.ts
index 28647077758..d5bf79a1ca5 100644
--- a/web-client/src/presenter/sequences/reviewExternalDocumentInformationSequence.ts
+++ b/web-client/src/presenter/sequences/reviewExternalDocumentInformationSequence.ts
@@ -3,7 +3,6 @@ import { generateEntryOfAppearancePdfAction } from '@web-client/presenter/action
 import { generateTitleAction } from '../actions/FileDocument/generateTitleAction';
 import { generateTitleForSupportingDocumentsAction } from '../actions/FileDocument/generateTitleForSupportingDocumentsAction';
 import { navigateToReviewFileADocumentAction } from '../actions/FileDocument/navigateToReviewFileADocumentAction';
-import { setAlertErrorAction } from '../actions/setAlertErrorAction';
 import { setFilersFromFilersMapAction } from '../actions/setFilersFromFilersMapAction';
 import { setPdfPreviewUrlAction } from '@web-client/presenter/actions/CourtIssuedOrder/setPdfPreviewUrlAction';
 import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
@@ -23,7 +22,6 @@ export const reviewExternalDocumentInformationSequence =
     validateExternalDocumentInformationAction,
     {
       error: [
-        setAlertErrorAction,
         setValidationErrorsAction,
         setScrollToErrorNotificationAction,
         setValidationAlertErrorsAction,
diff --git a/web-client/src/presenter/sequences/saveCourtIssuedDocketEntrySequence.ts b/web-client/src/presenter/sequences/saveCourtIssuedDocketEntrySequence.ts
index 7bc519947cd..66fad3585b5 100644
--- a/web-client/src/presenter/sequences/saveCourtIssuedDocketEntrySequence.ts
+++ b/web-client/src/presenter/sequences/saveCourtIssuedDocketEntrySequence.ts
@@ -1,8 +1,8 @@
 import { clearAlertsAction } from '../actions/clearAlertsAction';
 import { clearModalStateAction } from '../actions/clearModalStateAction';
 import { isDocketEntryMultiDocketableAction } from '../actions/CaseConsolidation/isDocketEntryMultiDocketableAction';
-import { setAlertErrorAction } from '../actions/setAlertErrorAction';
 import { setMultiDocketingCheckboxesAction } from '../actions/CaseConsolidation/setMultiDocketingCheckboxesAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
 import { setShowModalFactoryAction } from '../actions/setShowModalFactoryAction';
 import { setValidationAlertErrorsAction } from '../actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
@@ -21,8 +21,8 @@ export const saveCourtIssuedDocketEntrySequence = [
       validateCourtIssuedDocketEntryAction,
       {
         error: [
-          setAlertErrorAction,
           setValidationErrorsAction,
+          setScrollToErrorNotificationAction,
           setValidationAlertErrorsAction,
         ],
         success: [
diff --git a/web-client/src/presenter/sequences/saveSavedCaseForLaterSequence.ts b/web-client/src/presenter/sequences/saveSavedCaseForLaterSequence.ts
index 1bd0d441577..0c41ae8213d 100644
--- a/web-client/src/presenter/sequences/saveSavedCaseForLaterSequence.ts
+++ b/web-client/src/presenter/sequences/saveSavedCaseForLaterSequence.ts
@@ -4,13 +4,13 @@ import { filterEmptyStatisticsAction } from '../actions/StartCaseInternal/filter
 import { getPetitionIdAction } from '../actions/getPetitionIdAction';
 import { navigateToReviewSavedPetitionAction } from '../actions/CaseDetailEdit/navigateToReviewSavedPetitionAction';
 import { saveCaseDetailInternalEditAction } from '../actions/saveCaseDetailInternalEditAction';
-import { setAlertErrorAction } from '../actions/setAlertErrorAction';
 import { setCaseAction } from '../actions/setCaseAction';
 import { setCaseOnFormAction } from '../actions/setCaseOnFormAction';
 import { setCaseTypeAction } from '../actions/setCaseTypeAction';
 import { setContactsOnFormAction } from '../actions/setContactsOnFormAction';
 import { setDocketEntryIdAction } from '../actions/setDocketEntryIdAction';
 import { setProgressForFileUploadAction } from '@web-client/presenter/actions/setProgressForFileUploadAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
 import { setValidationAlertErrorsAction } from '../actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
 import { setupCurrentPageAction } from '../actions/setupCurrentPageAction';
@@ -28,8 +28,8 @@ export const saveSavedCaseForLaterSequence = showProgressSequenceDecorator([
   validateCaseDetailAction,
   {
     error: [
-      setAlertErrorAction,
       setValidationErrorsAction,
+      setScrollToErrorNotificationAction,
       setValidationAlertErrorsAction,
     ],
     success: [
diff --git a/web-client/src/presenter/sequences/selectDateRangeFromCalendarSequence.ts b/web-client/src/presenter/sequences/selectDateRangeFromCalendarSequence.ts
index 1328f60935a..078a942550b 100644
--- a/web-client/src/presenter/sequences/selectDateRangeFromCalendarSequence.ts
+++ b/web-client/src/presenter/sequences/selectDateRangeFromCalendarSequence.ts
@@ -1,5 +1,4 @@
 import { clearAlertsAction } from '../actions/clearAlertsAction';
-import { setAlertErrorAction } from '../actions/setAlertErrorAction';
 import { setValidationAlertErrorsAction } from '../actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
 import { shouldValidateAction } from '../actions/shouldValidateAction';
@@ -14,11 +13,7 @@ export const selectDateRangeFromCalendarSequence = [
     validate: [
       validateSearchDeadlinesAction,
       {
-        error: [
-          setAlertErrorAction,
-          setValidationErrorsAction,
-          setValidationAlertErrorsAction,
-        ],
+        error: [setValidationErrorsAction, setValidationAlertErrorsAction],
         success: [clearAlertsAction],
       },
     ],
diff --git a/web-client/src/presenter/sequences/setJudgeActivityReportFiltersSequence.ts b/web-client/src/presenter/sequences/setJudgeActivityReportFiltersSequence.ts
index fbf53f3917c..381484abfa3 100644
--- a/web-client/src/presenter/sequences/setJudgeActivityReportFiltersSequence.ts
+++ b/web-client/src/presenter/sequences/setJudgeActivityReportFiltersSequence.ts
@@ -1,5 +1,4 @@
 import { clearAlertsAction } from '../actions/clearAlertsAction';
-import { setAlertErrorAction } from '../actions/setAlertErrorAction';
 import { setJudgeActivityReportFiltersAction } from '../actions/JudgeActivityReport/setJudgeActivityReportFiltersAction';
 import { setValidationAlertErrorsAction } from '../actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
@@ -14,11 +13,7 @@ export const setJudgeActivityReportFiltersSequence = [
     validate: [
       validateJudgeActivityReportSearchAction,
       {
-        error: [
-          setAlertErrorAction,
-          setValidationErrorsAction,
-          setValidationAlertErrorsAction,
-        ],
+        error: [setValidationErrorsAction, setValidationAlertErrorsAction],
         success: [clearAlertsAction],
       },
     ],
diff --git a/web-client/src/presenter/sequences/submitAddDeficiencyStatisticsSequence.ts b/web-client/src/presenter/sequences/submitAddDeficiencyStatisticsSequence.ts
index c9f768667f1..22054b38fc5 100644
--- a/web-client/src/presenter/sequences/submitAddDeficiencyStatisticsSequence.ts
+++ b/web-client/src/presenter/sequences/submitAddDeficiencyStatisticsSequence.ts
@@ -5,6 +5,7 @@ import { navigateToCaseDetailCaseInformationActionFactory } from '../actions/nav
 import { setAlertErrorAction } from '../actions/setAlertErrorAction';
 import { setAlertSuccessAction } from '../actions/setAlertSuccessAction';
 import { setSaveAlertsForNavigationAction } from '../actions/setSaveAlertsForNavigationAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
 import { setValidationAlertErrorsAction } from '../actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
 import { showProgressSequenceDecorator } from '../utilities/showProgressSequenceDecorator';
@@ -18,8 +19,8 @@ export const submitAddDeficiencyStatisticsSequence = [
   validateAddDeficiencyStatisticsAction,
   {
     error: [
-      setAlertErrorAction,
       setValidationErrorsAction,
+      setScrollToErrorNotificationAction,
       setValidationAlertErrorsAction,
     ],
     success: [
diff --git a/web-client/src/presenter/sequences/submitAddPetitionerSequence.ts b/web-client/src/presenter/sequences/submitAddPetitionerSequence.ts
index d4bf5e1267e..4ad6c16e194 100644
--- a/web-client/src/presenter/sequences/submitAddPetitionerSequence.ts
+++ b/web-client/src/presenter/sequences/submitAddPetitionerSequence.ts
@@ -5,6 +5,7 @@ import { setAlertSuccessAction } from '../actions/setAlertSuccessAction';
 import { setCaseDetailPageTabFrozenAction } from '../actions/CaseDetail/setCaseDetailPageTabFrozenAction';
 import { setPartyViewTabAfterUpdatingPetitionersAction } from '../actions/setPartyViewTabAfterUpdatingPetitionersAction';
 import { setSaveAlertsForNavigationAction } from '../actions/setSaveAlertsForNavigationAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
 import { setValidationAlertErrorsAction } from '../actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
 import { setupCurrentPageAction } from '../actions/setupCurrentPageAction';
@@ -17,7 +18,11 @@ export const submitAddPetitionerSequence = [
   startShowValidationAction,
   validateAddPetitionerAction,
   {
-    error: [setValidationAlertErrorsAction, setValidationErrorsAction],
+    error: [
+      setValidationErrorsAction,
+      setScrollToErrorNotificationAction,
+      setValidationAlertErrorsAction,
+    ],
     success: showProgressSequenceDecorator([
       addPetitionerToCaseAction,
       setPartyViewTabAfterUpdatingPetitionersAction,
diff --git a/web-client/src/presenter/sequences/submitAddPractitionerDocumentSequence.ts b/web-client/src/presenter/sequences/submitAddPractitionerDocumentSequence.ts
index 808055a58ef..38775ed3aef 100644
--- a/web-client/src/presenter/sequences/submitAddPractitionerDocumentSequence.ts
+++ b/web-client/src/presenter/sequences/submitAddPractitionerDocumentSequence.ts
@@ -4,9 +4,9 @@ import { computeCategoryNameAction } from '../actions/computeCategoryNameAction'
 import { createPractitionerDocumentAction } from '../actions/createPractitionerDocumentAction';
 import { getUploadPractitionerDocumentAlertSuccessAction } from '../actions/Practitioners/getUploadPractitionerDocumentAlertSuccessAction';
 import { navigateToPractitionerDocumentsPageAction } from '../actions/navigateToPractitionerDocumentsPageAction';
-import { setAlertErrorAction } from '../actions/setAlertErrorAction';
 import { setAlertSuccessAction } from '../actions/setAlertSuccessAction';
 import { setSaveAlertsForNavigationAction } from '../actions/setSaveAlertsForNavigationAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
 import { setValidationAlertErrorsAction } from '../actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
 import { showProgressSequenceDecorator } from '../utilities/showProgressSequenceDecorator';
@@ -19,8 +19,8 @@ export const submitAddPractitionerDocumentSequence = [
   validateAddPractitionerDocumentFormAction,
   {
     error: [
-      setAlertErrorAction,
       setValidationErrorsAction,
+      setScrollToErrorNotificationAction,
       setValidationAlertErrorsAction,
     ],
     success: [
diff --git a/web-client/src/presenter/sequences/submitAddPractitionerSequence.ts b/web-client/src/presenter/sequences/submitAddPractitionerSequence.ts
index 2d047312d78..786bdc82b21 100644
--- a/web-client/src/presenter/sequences/submitAddPractitionerSequence.ts
+++ b/web-client/src/presenter/sequences/submitAddPractitionerSequence.ts
@@ -6,6 +6,7 @@ import { navigateToPractitionerDetailAction } from '../actions/navigateToPractit
 import { setAlertErrorAction } from '../actions/setAlertErrorAction';
 import { setAlertSuccessAction } from '../actions/setAlertSuccessAction';
 import { setSaveAlertsForNavigationAction } from '../actions/setSaveAlertsForNavigationAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
 import { setValidationAlertErrorsAction } from '../actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
 import { setWaitingForResponseAction } from '../actions/setWaitingForResponseAction';
@@ -32,7 +33,11 @@ export const submitAddPractitionerSequence = [
   startShowValidationAction,
   validateAddPractitionerAction,
   {
-    error: [setValidationErrorsAction, setValidationAlertErrorsAction],
+    error: [
+      setValidationErrorsAction,
+      setScrollToErrorNotificationAction,
+      setValidationAlertErrorsAction,
+    ],
     success: [
       setWaitingForResponseAction,
       hasUpdatedEmailFactoryAction('email'),
diff --git a/web-client/src/presenter/sequences/submitChangeLoginAndServiceEmailSequence.ts b/web-client/src/presenter/sequences/submitChangeLoginAndServiceEmailSequence.ts
index 6a509c9c123..ba073a3ce8a 100644
--- a/web-client/src/presenter/sequences/submitChangeLoginAndServiceEmailSequence.ts
+++ b/web-client/src/presenter/sequences/submitChangeLoginAndServiceEmailSequence.ts
@@ -1,6 +1,7 @@
 import { checkEmailAvailabilityAction } from '../actions/checkEmailAvailabilityAction';
 import { clearAlertsAction } from '../actions/clearAlertsAction';
 import { setAlertErrorAction } from '../actions/setAlertErrorAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
 import { setShowModalFactoryAction } from '../actions/setShowModalFactoryAction';
 import { setUserAction } from '../actions/setUserAction';
 import { setUserPermissionsAction } from '../actions/setUserPermissionsAction';
@@ -18,8 +19,8 @@ export const submitChangeLoginAndServiceEmailSequence = [
   validateChangeLoginAndServiceEmailAction,
   {
     error: [
-      setAlertErrorAction,
       setValidationErrorsAction,
+      setScrollToErrorNotificationAction,
       setValidationAlertErrorsAction,
     ],
     success: showProgressSequenceDecorator([
diff --git a/web-client/src/presenter/sequences/submitCourtIssuedDocketEntrySequence.ts b/web-client/src/presenter/sequences/submitCourtIssuedDocketEntrySequence.ts
index 069c0ecf858..7c39acc50aa 100644
--- a/web-client/src/presenter/sequences/submitCourtIssuedDocketEntrySequence.ts
+++ b/web-client/src/presenter/sequences/submitCourtIssuedDocketEntrySequence.ts
@@ -8,9 +8,9 @@ import { getDocketNumbersForConsolidatedServiceAction } from '../actions/getDock
 import { isCoversheetNeededAction } from '../actions/DocketEntry/isCoversheetNeededAction';
 import { isEditingDocketEntryAction } from '../actions/CourtIssuedDocketEntry/isEditingDocketEntryAction';
 import { navigateToCaseDetailAction } from '../actions/navigateToCaseDetailAction';
-import { setAlertErrorAction } from '../actions/setAlertErrorAction';
 import { setAlertSuccessAction } from '../actions/setAlertSuccessAction';
 import { setSaveAlertsForNavigationAction } from '../actions/setSaveAlertsForNavigationAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
 import { setValidationAlertErrorsAction } from '../actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
 import { shouldSaveToConsolidatedGroupAction } from '../actions/shouldSaveToConsolidatedGroupAction';
@@ -28,8 +28,8 @@ export const submitCourtIssuedDocketEntrySequence = [
   validateCourtIssuedDocketEntryAction,
   {
     error: [
-      setAlertErrorAction,
       setValidationErrorsAction,
+      setScrollToErrorNotificationAction,
       setValidationAlertErrorsAction,
     ],
     success: showProgressSequenceDecorator([
diff --git a/web-client/src/presenter/sequences/submitCourtIssuedOrderSequence.ts b/web-client/src/presenter/sequences/submitCourtIssuedOrderSequence.ts
index b7e295d3d84..36f193df37b 100644
--- a/web-client/src/presenter/sequences/submitCourtIssuedOrderSequence.ts
+++ b/web-client/src/presenter/sequences/submitCourtIssuedOrderSequence.ts
@@ -17,6 +17,7 @@ import { setAlertSuccessAction } from '../actions/setAlertSuccessAction';
 import { setCaseAction } from '../actions/setCaseAction';
 import { setDefaultDraftDocumentIdAction } from '../actions/setDefaultDraftDocumentIdAction';
 import { setSaveAlertsForNavigationAction } from '../actions/setSaveAlertsForNavigationAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
 import { setValidationAlertErrorsAction } from '../actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
 import { showProgressSequenceDecorator } from '../utilities/showProgressSequenceDecorator';
@@ -88,7 +89,11 @@ export const submitCourtIssuedOrderSequence = showProgressSequenceDecorator([
       startShowValidationAction,
       validateCourtOrderAction,
       {
-        error: [setValidationErrorsAction, setValidationAlertErrorsAction],
+        error: [
+          setValidationErrorsAction,
+          setScrollToErrorNotificationAction,
+          setValidationAlertErrorsAction,
+        ],
         success: submitCourtIssuedOrder,
       },
     ],
diff --git a/web-client/src/presenter/sequences/submitEditContactSequence.ts b/web-client/src/presenter/sequences/submitEditContactSequence.ts
index 6c23085080a..413822691ad 100644
--- a/web-client/src/presenter/sequences/submitEditContactSequence.ts
+++ b/web-client/src/presenter/sequences/submitEditContactSequence.ts
@@ -3,7 +3,9 @@ import { navigateToCaseDetailAction } from '../actions/navigateToCaseDetailActio
 import { setAlertSuccessAction } from '../actions/setAlertSuccessAction';
 import { setCaseDetailPageTabFrozenAction } from '../actions/CaseDetail/setCaseDetailPageTabFrozenAction';
 import { setSaveAlertsForNavigationAction } from '../actions/setSaveAlertsForNavigationAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
 import { setValidationAlertErrorsAction } from '../actions/setValidationAlertErrorsAction';
+import { setValidationErrorsAction } from '@web-client/presenter/actions/setValidationErrorsAction';
 import { setupCurrentPageAction } from '../actions/setupCurrentPageAction';
 import { showProgressSequenceDecorator } from '../utilities/showProgressSequenceDecorator';
 import { startShowValidationAction } from '../actions/startShowValidationAction';
@@ -15,7 +17,11 @@ export const submitEditContactSequence = [
   startShowValidationAction,
   validatePetitionerAction,
   {
-    error: [setValidationAlertErrorsAction],
+    error: [
+      setValidationErrorsAction,
+      setScrollToErrorNotificationAction,
+      setValidationAlertErrorsAction,
+    ],
     success: showProgressSequenceDecorator([
       updateContactAction,
       setAlertSuccessAction,
diff --git a/web-client/src/presenter/sequences/submitEditDeficiencyStatisticSequence.ts b/web-client/src/presenter/sequences/submitEditDeficiencyStatisticSequence.ts
index f1a92f0f8b9..70667efcb30 100644
--- a/web-client/src/presenter/sequences/submitEditDeficiencyStatisticSequence.ts
+++ b/web-client/src/presenter/sequences/submitEditDeficiencyStatisticSequence.ts
@@ -6,6 +6,7 @@ import { setAlertErrorAction } from '../actions/setAlertErrorAction';
 import { setAlertSuccessAction } from '../actions/setAlertSuccessAction';
 import { setCaseDetailPageTabFrozenAction } from '../actions/CaseDetail/setCaseDetailPageTabFrozenAction';
 import { setSaveAlertsForNavigationAction } from '../actions/setSaveAlertsForNavigationAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
 import { setValidationAlertErrorsAction } from '../actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
 import { showProgressSequenceDecorator } from '../utilities/showProgressSequenceDecorator';
@@ -19,8 +20,8 @@ export const submitEditDeficiencyStatisticSequence = [
   validateAddDeficiencyStatisticsAction,
   {
     error: [
-      setAlertErrorAction,
       setValidationErrorsAction,
+      setScrollToErrorNotificationAction,
       setValidationAlertErrorsAction,
     ],
     success: [
diff --git a/web-client/src/presenter/sequences/submitEditPetitionerCounselSequence.ts b/web-client/src/presenter/sequences/submitEditPetitionerCounselSequence.ts
index a3e2fabc66e..5c750c3296b 100644
--- a/web-client/src/presenter/sequences/submitEditPetitionerCounselSequence.ts
+++ b/web-client/src/presenter/sequences/submitEditPetitionerCounselSequence.ts
@@ -7,7 +7,9 @@ import { setAlertSuccessAction } from '../actions/setAlertSuccessAction';
 import { setCaseAction } from '../actions/setCaseAction';
 import { setCasePropFromStateAction } from '../actions/setCasePropFromStateAction';
 import { setRepresentingFromRepresentingMapActionFactory } from '../actions/setRepresentingFromRepresentingMapActionFactory';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
 import { setValidationAlertErrorsAction } from '../actions/setValidationAlertErrorsAction';
+import { setValidationErrorsAction } from '@web-client/presenter/actions/setValidationErrorsAction';
 import { showProgressSequenceDecorator } from '../utilities/showProgressSequenceDecorator';
 import { startShowValidationAction } from '../actions/startShowValidationAction';
 import { stopShowValidationAction } from '../actions/stopShowValidationAction';
@@ -19,7 +21,11 @@ export const submitEditPetitionerCounselSequence = [
   setRepresentingFromRepresentingMapActionFactory('form'),
   validateEditPetitionerCounselAction,
   {
-    error: [setValidationAlertErrorsAction],
+    error: [
+      setValidationErrorsAction,
+      setScrollToErrorNotificationAction,
+      setValidationAlertErrorsAction,
+    ],
     success: showProgressSequenceDecorator([
       clearAlertsAction,
       stopShowValidationAction,
diff --git a/web-client/src/presenter/sequences/submitEditPetitionerSequence.ts b/web-client/src/presenter/sequences/submitEditPetitionerSequence.ts
index c3082f2e3c5..56f652a508f 100644
--- a/web-client/src/presenter/sequences/submitEditPetitionerSequence.ts
+++ b/web-client/src/presenter/sequences/submitEditPetitionerSequence.ts
@@ -1,7 +1,9 @@
 import { clearAlertsAction } from '../actions/clearAlertsAction';
 import { hasUpdatedPetitionerEmailAction } from '../actions/hasUpdatedPetitionerEmailAction';
 import { openGainElectronicAccessToCaseModalSequence } from './openGainElectronicAccessToCaseModalSequence';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
 import { setValidationAlertErrorsAction } from '../actions/setValidationAlertErrorsAction';
+import { setValidationErrorsAction } from '@web-client/presenter/actions/setValidationErrorsAction';
 import { showProgressSequenceDecorator } from '../utilities/showProgressSequenceDecorator';
 import { startShowValidationAction } from '../actions/startShowValidationAction';
 import { submitUpdatePetitionerInformationSequence } from './submitUpdatePetitionerInformationSequence';
@@ -12,7 +14,11 @@ export const submitEditPetitionerSequence = [
   startShowValidationAction,
   validatePetitionerAction,
   {
-    error: [setValidationAlertErrorsAction],
+    error: [
+      setValidationErrorsAction,
+      setScrollToErrorNotificationAction,
+      setValidationAlertErrorsAction,
+    ],
     success: showProgressSequenceDecorator([
       hasUpdatedPetitionerEmailAction,
       {
diff --git a/web-client/src/presenter/sequences/submitEditPractitionerDocumentSequence.ts b/web-client/src/presenter/sequences/submitEditPractitionerDocumentSequence.ts
index 61290c86b18..b870d372ee5 100644
--- a/web-client/src/presenter/sequences/submitEditPractitionerDocumentSequence.ts
+++ b/web-client/src/presenter/sequences/submitEditPractitionerDocumentSequence.ts
@@ -4,9 +4,9 @@ import { computeCategoryNameAction } from '../actions/computeCategoryNameAction'
 import { editPractitionerDocumentAction } from '../actions/editPractitionerDocumentAction';
 import { getEditPractitionerDocumentAlertSuccessAction } from '../actions/Practitioners/getEditPractitionerDocumentAlertSuccessAction';
 import { navigateToPractitionerDocumentsPageAction } from '../actions/navigateToPractitionerDocumentsPageAction';
-import { setAlertErrorAction } from '../actions/setAlertErrorAction';
 import { setAlertSuccessAction } from '../actions/setAlertSuccessAction';
 import { setSaveAlertsForNavigationAction } from '../actions/setSaveAlertsForNavigationAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
 import { setValidationAlertErrorsAction } from '../actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
 import { showProgressSequenceDecorator } from '../utilities/showProgressSequenceDecorator';
@@ -19,8 +19,8 @@ export const submitEditPractitionerDocumentSequence = [
   validateAddPractitionerDocumentFormAction,
   {
     error: [
-      setAlertErrorAction,
       setValidationErrorsAction,
+      setScrollToErrorNotificationAction,
       setValidationAlertErrorsAction,
     ],
     success: [
diff --git a/web-client/src/presenter/sequences/submitEditRespondentCounselSequence.ts b/web-client/src/presenter/sequences/submitEditRespondentCounselSequence.ts
index 898be27c80b..200750039cb 100644
--- a/web-client/src/presenter/sequences/submitEditRespondentCounselSequence.ts
+++ b/web-client/src/presenter/sequences/submitEditRespondentCounselSequence.ts
@@ -5,7 +5,9 @@ import { navigateToCaseDetailCaseInformationActionFactory } from '../actions/nav
 import { setAlertSuccessAction } from '../actions/setAlertSuccessAction';
 import { setCaseAction } from '../actions/setCaseAction';
 import { setCasePropFromStateAction } from '../actions/setCasePropFromStateAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
 import { setValidationAlertErrorsAction } from '../actions/setValidationAlertErrorsAction';
+import { setValidationErrorsAction } from '@web-client/presenter/actions/setValidationErrorsAction';
 import { showProgressSequenceDecorator } from '../utilities/showProgressSequenceDecorator';
 import { startShowValidationAction } from '../actions/startShowValidationAction';
 import { submitEditRespondentCounselAction } from '../actions/CaseAssociation/submitEditRespondentCounselAction';
@@ -15,7 +17,11 @@ export const submitEditRespondentCounselSequence = [
   startShowValidationAction,
   validateEditRespondentCounselAction,
   {
-    error: [setValidationAlertErrorsAction],
+    error: [
+      setValidationErrorsAction,
+      setScrollToErrorNotificationAction,
+      setValidationAlertErrorsAction,
+    ],
     success: showProgressSequenceDecorator([
       submitEditRespondentCounselAction,
       {
diff --git a/web-client/src/presenter/sequences/submitStampMotionSequence.ts b/web-client/src/presenter/sequences/submitStampMotionSequence.ts
index 6aa9fd168d4..ac44245eeb4 100644
--- a/web-client/src/presenter/sequences/submitStampMotionSequence.ts
+++ b/web-client/src/presenter/sequences/submitStampMotionSequence.ts
@@ -4,12 +4,12 @@ import { completeMotionStampingAction } from '../actions/completeMotionStampingA
 import { followRedirectAction } from '../actions/followRedirectAction';
 import { getDraftDocumentTitleFromStampDataAction } from '../actions/StampMotion/getDraftDocumentTitleFromStampDataAction';
 import { navigateToDraftDocumentsAction } from '../actions/navigateToDraftDocumentsAction';
-import { setAlertErrorAction } from '../actions/setAlertErrorAction';
 import { setAlertSuccessAction } from '../actions/setAlertSuccessAction';
 import { setCaseDetailPageTabActionGenerator } from '../actions/setCaseDetailPageTabActionGenerator';
 import { setDefaultDraftDocumentIdAction } from '../actions/setDefaultDraftDocumentIdAction';
 import { setRedirectUrlAction } from '../actions/setRedirectUrlAction';
 import { setSaveAlertsForNavigationAction } from '../actions/setSaveAlertsForNavigationAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
 import { setSuccessfulStampFromDocumentTitleAction } from '../actions/StampMotion/setSuccessfulStampFromDocumentTitleAction';
 import { setValidationAlertErrorsAction } from '../actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
@@ -22,8 +22,8 @@ export const submitStampMotionSequence = showProgressSequenceDecorator([
   validateStampAction,
   {
     error: [
-      setAlertErrorAction,
       setValidationErrorsAction,
+      setScrollToErrorNotificationAction,
       setValidationAlertErrorsAction,
     ],
     success: [
diff --git a/web-client/src/presenter/sequences/submitTrialSessionSequence.ts b/web-client/src/presenter/sequences/submitTrialSessionSequence.ts
index 2168cbb6d8d..fab941fe7a6 100644
--- a/web-client/src/presenter/sequences/submitTrialSessionSequence.ts
+++ b/web-client/src/presenter/sequences/submitTrialSessionSequence.ts
@@ -22,7 +22,6 @@ export const submitTrialSessionSequence = [
   validateTrialSessionAction,
   {
     error: [
-      setAlertErrorAction,
       setValidationErrorsAction,
       setScrollToErrorNotificationAction,
       setValidationAlertErrorsAction,
diff --git a/web-client/src/presenter/sequences/submitUpdatePractitionerUserSequence.ts b/web-client/src/presenter/sequences/submitUpdatePractitionerUserSequence.ts
index 1360876fa18..947429eaa5f 100644
--- a/web-client/src/presenter/sequences/submitUpdatePractitionerUserSequence.ts
+++ b/web-client/src/presenter/sequences/submitUpdatePractitionerUserSequence.ts
@@ -3,6 +3,7 @@ import { clearAlertsAction } from '../actions/clearAlertsAction';
 import { clearScreenMetadataAction } from '../actions/clearScreenMetadataAction';
 import { hasUpdatedEmailFactoryAction } from '../actions/hasUpdatedEmailFactoryAction';
 import { setAlertErrorAction } from '../actions/setAlertErrorAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
 import { setValidationAlertErrorsAction } from '../actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
 import { setWaitingForResponseAction } from '../actions/setWaitingForResponseAction';
@@ -38,8 +39,8 @@ export const submitUpdatePractitionerUserSequence = [
             emailInUse: [
               unsetWaitingForResponseAction,
               clearAlertsAction,
-              setAlertErrorAction,
               setValidationErrorsAction,
+              setScrollToErrorNotificationAction,
               setValidationAlertErrorsAction,
               stopShowValidationAction,
             ],
diff --git a/web-client/src/presenter/sequences/submitUpdateUserContactInformationSequence.ts b/web-client/src/presenter/sequences/submitUpdateUserContactInformationSequence.ts
index ddd76727984..fed1a76b640 100644
--- a/web-client/src/presenter/sequences/submitUpdateUserContactInformationSequence.ts
+++ b/web-client/src/presenter/sequences/submitUpdateUserContactInformationSequence.ts
@@ -1,4 +1,5 @@
 import { clearAlertsAction } from '../actions/clearAlertsAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
 import { setValidationAlertErrorsAction } from '../actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
 import { setWaitingForResponseAction } from '../actions/setWaitingForResponseAction';
@@ -12,7 +13,11 @@ export const submitUpdateUserContactInformationSequence = [
   startShowValidationAction,
   validateUserContactAction,
   {
-    error: [setValidationErrorsAction, setValidationAlertErrorsAction],
+    error: [
+      setValidationErrorsAction,
+      setScrollToErrorNotificationAction,
+      setValidationAlertErrorsAction,
+    ],
     success: [
       setWaitingForResponseAction,
       setupCurrentPageAction('Interstitial'),
diff --git a/web-client/src/presenter/sequences/updateDateRangeForDeadlinesSequence.ts b/web-client/src/presenter/sequences/updateDateRangeForDeadlinesSequence.ts
index 3140e2093ba..850b51ee8b5 100644
--- a/web-client/src/presenter/sequences/updateDateRangeForDeadlinesSequence.ts
+++ b/web-client/src/presenter/sequences/updateDateRangeForDeadlinesSequence.ts
@@ -1,7 +1,6 @@
 import { clearAlertsAction } from '../actions/clearAlertsAction';
 import { clearCaseDeadlineReportAction } from '../actions/CaseDeadline/clearCaseDeadlineReportAction';
 import { getCaseDeadlinesAction } from '../actions/CaseDeadline/getCaseDeadlinesAction';
-import { setAlertErrorAction } from '../actions/setAlertErrorAction';
 import { setCaseDeadlinesAction } from '../actions/CaseDeadline/setCaseDeadlinesAction';
 import { setValidationAlertErrorsAction } from '../actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
@@ -17,7 +16,6 @@ export const updateDateRangeForDeadlinesSequence =
     {
       error: [
         startShowValidationAction,
-        setAlertErrorAction,
         setValidationErrorsAction,
         setValidationAlertErrorsAction,
       ],
diff --git a/web-client/src/presenter/sequences/updateTrialSessionSequence.ts b/web-client/src/presenter/sequences/updateTrialSessionSequence.ts
index 3fce51f9aff..5f71e1c2b50 100644
--- a/web-client/src/presenter/sequences/updateTrialSessionSequence.ts
+++ b/web-client/src/presenter/sequences/updateTrialSessionSequence.ts
@@ -2,6 +2,7 @@ import { clearAlertsAction } from '../actions/clearAlertsAction';
 import { clearPdfPreviewUrlAction } from '../actions/CourtIssuedOrder/clearPdfPreviewUrlAction';
 import { computeSubmitTrialSessionDataAction } from '../actions/TrialSession/computeSubmitTrialSessionDataAction';
 import { setAlertErrorAction } from '../actions/setAlertErrorAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
 import { setValidationAlertErrorsAction } from '../actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
 import { setWaitingForResponseAction } from '../actions/setWaitingForResponseAction';
@@ -18,8 +19,8 @@ export const updateTrialSessionSequence = [
   validateTrialSessionAction,
   {
     error: [
-      setAlertErrorAction,
       setValidationErrorsAction,
+      setScrollToErrorNotificationAction,
       setValidationAlertErrorsAction,
     ],
     success: [
diff --git a/web-client/src/presenter/sequences/validateAddDeficiencyStatisticsSequence.ts b/web-client/src/presenter/sequences/validateAddDeficiencyStatisticsSequence.ts
index 9a6d306f785..40495c69db7 100644
--- a/web-client/src/presenter/sequences/validateAddDeficiencyStatisticsSequence.ts
+++ b/web-client/src/presenter/sequences/validateAddDeficiencyStatisticsSequence.ts
@@ -1,5 +1,4 @@
 import { clearAlertsAction } from '../actions/clearAlertsAction';
-import { setAlertErrorAction } from '../actions/setAlertErrorAction';
 import { setValidationAlertErrorsAction } from '../actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
 import { shouldValidateAction } from '../actions/shouldValidateAction';
@@ -12,11 +11,7 @@ export const validateAddDeficiencyStatisticsSequence = [
     validate: [
       validateAddDeficiencyStatisticsAction,
       {
-        error: [
-          setAlertErrorAction,
-          setValidationErrorsAction,
-          setValidationAlertErrorsAction,
-        ],
+        error: [setValidationErrorsAction, setValidationAlertErrorsAction],
         success: [clearAlertsAction],
       },
     ],
diff --git a/web-client/src/presenter/sequences/validatePetitionerSequence.ts b/web-client/src/presenter/sequences/validatePetitionerSequence.ts
index 43f5c1c0de6..5aa2dbef4f8 100644
--- a/web-client/src/presenter/sequences/validatePetitionerSequence.ts
+++ b/web-client/src/presenter/sequences/validatePetitionerSequence.ts
@@ -1,18 +1,19 @@
+import { clearAlertsAction } from '@web-client/presenter/actions/clearAlertsAction';
+import { setValidationAlertErrorsAction } from '@web-client/presenter/actions/setValidationAlertErrorsAction';
+import { setValidationErrorsAction } from '@web-client/presenter/actions/setValidationErrorsAction';
 import { shouldValidateAction } from '../actions/shouldValidateAction';
-import { startShowValidationAction } from '../actions/startShowValidationAction';
-import { stopShowValidationAction } from '../actions/stopShowValidationAction';
 import { validatePetitionerAction } from '../actions/validatePetitionerAction';
 
 export const validatePetitionerSequence = [
-  stopShowValidationAction,
+  () => console.log('Running validation'),
   shouldValidateAction,
   {
     ignore: [],
     validate: [
       validatePetitionerAction,
       {
-        error: [startShowValidationAction],
-        success: [],
+        error: [setValidationErrorsAction, setValidationAlertErrorsAction],
+        success: [clearAlertsAction],
       },
     ],
   },
diff --git a/web-client/src/presenter/sequences/validateStampSequence.ts b/web-client/src/presenter/sequences/validateStampSequence.ts
index 7cf7c2a6632..e09f67d7f7c 100644
--- a/web-client/src/presenter/sequences/validateStampSequence.ts
+++ b/web-client/src/presenter/sequences/validateStampSequence.ts
@@ -1,5 +1,4 @@
 import { clearAlertsAction } from '../actions/clearAlertsAction';
-import { setAlertErrorAction } from '../actions/setAlertErrorAction';
 import { setValidationAlertErrorsAction } from '../actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
 import { shouldValidateAction } from '@web-client/presenter/actions/shouldValidateAction';
@@ -12,11 +11,7 @@ export const validateStampSequence = [
     validate: [
       validateStampAction,
       {
-        error: [
-          setAlertErrorAction,
-          setValidationErrorsAction,
-          setValidationAlertErrorsAction,
-        ],
+        error: [setValidationErrorsAction, setValidationAlertErrorsAction],
         success: [clearAlertsAction],
       },
     ],
diff --git a/web-client/src/presenter/sequences/validateUserContactSequence.ts b/web-client/src/presenter/sequences/validateUserContactSequence.ts
index 956107e6212..3f9f868f4bb 100644
--- a/web-client/src/presenter/sequences/validateUserContactSequence.ts
+++ b/web-client/src/presenter/sequences/validateUserContactSequence.ts
@@ -1,4 +1,5 @@
 import { clearAlertsAction } from '../actions/clearAlertsAction';
+import { setValidationAlertErrorsAction } from '@web-client/presenter/actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
 import { shouldValidateAction } from '../actions/shouldValidateAction';
 import { validateUserContactAction } from '../actions/validateUserContactAction';
@@ -10,7 +11,7 @@ export const validateUserContactSequence = [
     validate: [
       validateUserContactAction,
       {
-        error: [setValidationErrorsAction],
+        error: [setValidationErrorsAction, setValidationAlertErrorsAction],
         success: [clearAlertsAction],
       },
     ],
diff --git a/web-client/src/views/ContactEdit.tsx b/web-client/src/views/ContactEdit.tsx
index f5af3ed7032..020437937b9 100644
--- a/web-client/src/views/ContactEdit.tsx
+++ b/web-client/src/views/ContactEdit.tsx
@@ -37,7 +37,7 @@ export const ContactEdit = connect(
   }) {
     const type = 'contact';
     const bind = 'form';
-    const onBlur = 'validatePetitionerSequence';
+    const onBlur = () => validatePetitionerSequence();
 
     return (
       <>
@@ -77,6 +77,7 @@ export const ContactEdit = connect(
               bind={bind}
               clearTypeOnCountryChange={true}
               type={type}
+              onBlur={onBlur}
               onChange="contactPrimaryCountryTypeChangeSequence"
             />
             {form.contact.countryType === COUNTRY_TYPES.DOMESTIC ? (
diff --git a/web-client/src/views/EditPetitionerInformationInternal.tsx b/web-client/src/views/EditPetitionerInformationInternal.tsx
index a54da707cbd..bfac3ccde58 100644
--- a/web-client/src/views/EditPetitionerInformationInternal.tsx
+++ b/web-client/src/views/EditPetitionerInformationInternal.tsx
@@ -55,7 +55,7 @@ export const EditPetitionerInformationInternal = connect(
   }) {
     const type = 'contact';
     const bind = 'form';
-    const onBlur = validatePetitionerSequence;
+    const onBlur = () => validatePetitionerSequence();
 
     return (
       <>
@@ -155,6 +155,7 @@ export const EditPetitionerInformationInternal = connect(
               bind={bind}
               clearTypeOnCountryChange={true}
               type={type}
+              onBlur={onBlur}
               onChange="updateFormValueSequence"
             />
             {form.contact.countryType === COUNTRY_TYPES.DOMESTIC ? (
@@ -266,11 +267,7 @@ export const EditPetitionerInformationInternal = connect(
               !form.contact.currentEmail && (
                 <>
                   <h4>Add Login & Service Email</h4>
-                  <FormGroup
-                    errorText={
-                      validationErrors.contact && validationErrors.contact.email
-                    }
-                  >
+                  <FormGroup errorText={validationErrors?.contact?.email}>
                     <label className="usa-label" htmlFor="updatedEmail">
                       New email address
                     </label>
@@ -292,10 +289,7 @@ export const EditPetitionerInformationInternal = connect(
                     />
                   </FormGroup>
                   <FormGroup
-                    errorText={
-                      validationErrors.contact &&
-                      validationErrors.contact.confirmEmail
-                    }
+                    errorText={validationErrors?.contact?.confirmEmail}
                   >
                     <label className="usa-label" htmlFor="confirm-email">
                       Re-enter new email address

From 5fa5ee134ba8e4c4ce5f11d3526ca799c87373b1 Mon Sep 17 00:00:00 2001
From: Christopher Bisom <cbisom@flexion.us>
Date: Fri, 20 Sep 2024 17:22:52 -0400
Subject: [PATCH 22/33] 10074-bug: add scroll to some report sequences

---
 .../presenter/sequences/getCustomCaseReportSequence.ts   | 9 +++++++--
 .../sequences/updateDateRangeForDeadlinesSequence.ts     | 2 ++
 .../presenter/sequences/validatePetitionerSequence.ts    | 1 -
 3 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/web-client/src/presenter/sequences/getCustomCaseReportSequence.ts b/web-client/src/presenter/sequences/getCustomCaseReportSequence.ts
index 0a521f01240..2e5e9e96009 100644
--- a/web-client/src/presenter/sequences/getCustomCaseReportSequence.ts
+++ b/web-client/src/presenter/sequences/getCustomCaseReportSequence.ts
@@ -1,7 +1,8 @@
 import { clearAlertsAction } from '../actions/clearAlertsAction';
 import { clearErrorAlertsAction } from '../actions/clearErrorAlertsAction';
 import { getCustomCaseReportAction } from '../actions/CaseInventoryReport/getCustomCaseReportAction';
-import { setAlertErrorAction } from '../actions/setAlertErrorAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
+import { setValidationAlertErrorsAction } from '@web-client/presenter/actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
 import { setWaitingForResponseAction } from '../actions/setWaitingForResponseAction';
 import { startShowValidationAction } from '../actions/startShowValidationAction';
@@ -14,7 +15,11 @@ export const getCustomCaseReportSequence = [
   startShowValidationAction,
   validateCustomCaseReportFiltersAction,
   {
-    error: [setAlertErrorAction, setValidationErrorsAction],
+    error: [
+      setValidationErrorsAction,
+      setScrollToErrorNotificationAction,
+      setValidationAlertErrorsAction,
+    ],
     success: [
       stopShowValidationAction,
       clearErrorAlertsAction,
diff --git a/web-client/src/presenter/sequences/updateDateRangeForDeadlinesSequence.ts b/web-client/src/presenter/sequences/updateDateRangeForDeadlinesSequence.ts
index 850b51ee8b5..b09c367a30b 100644
--- a/web-client/src/presenter/sequences/updateDateRangeForDeadlinesSequence.ts
+++ b/web-client/src/presenter/sequences/updateDateRangeForDeadlinesSequence.ts
@@ -2,6 +2,7 @@ import { clearAlertsAction } from '../actions/clearAlertsAction';
 import { clearCaseDeadlineReportAction } from '../actions/CaseDeadline/clearCaseDeadlineReportAction';
 import { getCaseDeadlinesAction } from '../actions/CaseDeadline/getCaseDeadlinesAction';
 import { setCaseDeadlinesAction } from '../actions/CaseDeadline/setCaseDeadlinesAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
 import { setValidationAlertErrorsAction } from '../actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
 import { showProgressSequenceDecorator } from '../utilities/showProgressSequenceDecorator';
@@ -17,6 +18,7 @@ export const updateDateRangeForDeadlinesSequence =
       error: [
         startShowValidationAction,
         setValidationErrorsAction,
+        setScrollToErrorNotificationAction,
         setValidationAlertErrorsAction,
       ],
       success: [
diff --git a/web-client/src/presenter/sequences/validatePetitionerSequence.ts b/web-client/src/presenter/sequences/validatePetitionerSequence.ts
index 5aa2dbef4f8..01a0b52394a 100644
--- a/web-client/src/presenter/sequences/validatePetitionerSequence.ts
+++ b/web-client/src/presenter/sequences/validatePetitionerSequence.ts
@@ -5,7 +5,6 @@ import { shouldValidateAction } from '../actions/shouldValidateAction';
 import { validatePetitionerAction } from '../actions/validatePetitionerAction';
 
 export const validatePetitionerSequence = [
-  () => console.log('Running validation'),
   shouldValidateAction,
   {
     ignore: [],

From 6d2a075f1e007732f67d8cb19854f812393bde53 Mon Sep 17 00:00:00 2001
From: Christopher Bisom <cbisom@flexion.us>
Date: Mon, 23 Sep 2024 08:07:19 -0400
Subject: [PATCH 23/33] 10074: fix error as well as a type; add a test

---
 .../setValidationAlertErrorsAction.test.ts    | 20 +++++++++++++++++++
 .../views/CaseDetail/PetitionPaymentForm.tsx  |  2 +-
 .../src/views/StartCase/CaseTypeSelect.tsx    |  2 +-
 3 files changed, 22 insertions(+), 2 deletions(-)

diff --git a/web-client/src/presenter/actions/setValidationAlertErrorsAction.test.ts b/web-client/src/presenter/actions/setValidationAlertErrorsAction.test.ts
index 1882ab0fbf2..f4b05f4e5e3 100644
--- a/web-client/src/presenter/actions/setValidationAlertErrorsAction.test.ts
+++ b/web-client/src/presenter/actions/setValidationAlertErrorsAction.test.ts
@@ -17,6 +17,7 @@ describe('setValidationAlertErrors', () => {
     });
     expect(state.alertError).toMatchObject({
       messages: ['Some issue occurred'],
+      scrollToErrorNotification: false,
     });
   });
 
@@ -140,4 +141,23 @@ describe('setValidationAlertErrors', () => {
       messages: ['second nested', 'first nested'],
     });
   });
+
+  it('state.alertError sets scrollToErrorNotification appropriately', async () => {
+    const { state } = await runAction(setValidationAlertErrorsAction, {
+      modules: {
+        presenter,
+      },
+      props: {
+        errors: {
+          irsNoticeDate: 'Some issue occurred',
+        },
+        scrollToErrorNotification: true,
+      },
+      state: {},
+    });
+    expect(state.alertError).toMatchObject({
+      messages: ['Some issue occurred'],
+      scrollToErrorNotification: true,
+    });
+  });
 });
diff --git a/web-client/src/views/CaseDetail/PetitionPaymentForm.tsx b/web-client/src/views/CaseDetail/PetitionPaymentForm.tsx
index 5fbba8ff003..cadfa2acee9 100644
--- a/web-client/src/views/CaseDetail/PetitionPaymentForm.tsx
+++ b/web-client/src/views/CaseDetail/PetitionPaymentForm.tsx
@@ -139,7 +139,7 @@ const PetitionPaymentFormComponent: React.FC<ComponentProps & any> = ({
               id="petition-payment-method"
               name="petitionPaymentMethod"
               value={bind.petitionPaymentMethod || ''}
-              onBlur={validateFormData}
+              onBlur={() => validateFormData()}
               onChange={e => {
                 onUpdate({
                   key: e.target.name,
diff --git a/web-client/src/views/StartCase/CaseTypeSelect.tsx b/web-client/src/views/StartCase/CaseTypeSelect.tsx
index dd60e3641f1..2686f249b13 100644
--- a/web-client/src/views/StartCase/CaseTypeSelect.tsx
+++ b/web-client/src/views/StartCase/CaseTypeSelect.tsx
@@ -71,7 +71,7 @@ export const CaseTypeSelect = connect<
               name={name || 'caseType'}
               ref={refProp}
               value={value}
-              onBlur={onBlur}
+              onBlur={onBlur ? () => onBlur() : undefined}
               onChange={e => {
                 onChange({
                   key: e.target.name,

From de8d665be2d0ac8ab36feca09d06a9e288893fa4 Mon Sep 17 00:00:00 2001
From: Christopher Bisom <cbisom@flexion.us>
Date: Mon, 23 Sep 2024 08:42:44 -0400
Subject: [PATCH 24/33] 10074-bug: fix a couple of breaking tests

---
 shared/src/business/entities/cases/Case.ts                | 8 ++++----
 .../presenter/actions/validatePetitionerAction.test.ts    | 2 +-
 .../src/views/Practitioners/PractitionerContactForm.tsx   | 2 +-
 web-client/src/views/StartCaseInternal/StatisticsForm.tsx | 2 +-
 4 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/shared/src/business/entities/cases/Case.ts b/shared/src/business/entities/cases/Case.ts
index 128d485146f..edfb56a8ad9 100644
--- a/shared/src/business/entities/cases/Case.ts
+++ b/shared/src/business/entities/cases/Case.ts
@@ -534,7 +534,7 @@ export class Case extends JoiValidationEntity {
     mailingDate: JoiValidationConstants.STRING.max(25)
       .when('isPaper', {
         is: true,
-        otherwise: joi.allow(null).optional(),
+        otherwise: joi.optional().allow(null),
         then: joi.required(),
       })
       .description('Date that petition was mailed to the court.')
@@ -590,7 +590,7 @@ export class Case extends JoiValidationEntity {
       'petitionPaymentStatus',
       {
         is: PAYMENT_STATUS.PAID,
-        otherwise: joi.allow(null).optional(),
+        otherwise: joi.optional().allow(null),
         then: JoiValidationConstants.ISO_DATE.max('now').required(),
       },
     )
@@ -599,7 +599,7 @@ export class Case extends JoiValidationEntity {
     petitionPaymentMethod: JoiValidationConstants.STRING.max(50)
       .when('petitionPaymentStatus', {
         is: PAYMENT_STATUS.PAID,
-        otherwise: joi.allow(null).optional(),
+        otherwise: joi.optional().allow(null),
         then: joi.required(),
       })
       .description('How the petitioner paid the case fee.')
@@ -617,7 +617,7 @@ export class Case extends JoiValidationEntity {
       'petitionPaymentStatus',
       {
         is: PAYMENT_STATUS.WAIVED,
-        otherwise: joi.allow(null).optional(),
+        otherwise: joi.optional().allow(null),
         then: JoiValidationConstants.ISO_DATE.max('now').required(),
       },
     )
diff --git a/web-client/src/presenter/actions/validatePetitionerAction.test.ts b/web-client/src/presenter/actions/validatePetitionerAction.test.ts
index 6d7c3609e9f..ecf6ed7c59a 100644
--- a/web-client/src/presenter/actions/validatePetitionerAction.test.ts
+++ b/web-client/src/presenter/actions/validatePetitionerAction.test.ts
@@ -40,7 +40,7 @@ describe('validatePetitionerAction', () => {
       },
     });
 
-    expect(result.state.validationErrors.contact).toEqual(null);
+    expect(result.state.validationErrors).toBeUndefined();
     expect(successStub.mock.calls.length).toEqual(1);
   });
 
diff --git a/web-client/src/views/Practitioners/PractitionerContactForm.tsx b/web-client/src/views/Practitioners/PractitionerContactForm.tsx
index 93617163333..e2f0e2512a1 100644
--- a/web-client/src/views/Practitioners/PractitionerContactForm.tsx
+++ b/web-client/src/views/Practitioners/PractitionerContactForm.tsx
@@ -69,7 +69,7 @@ export const PractitionerContactForm = connect(
                 name="contact.phone"
                 type="text"
                 value={form.contact.phone || ''}
-                onBlur={onBlurValidationSequence}
+                onBlur={() => onBlurValidationSequence()}
                 onChange={e => {
                   onChangeUpdateSequence({
                     key: e.target.name,
diff --git a/web-client/src/views/StartCaseInternal/StatisticsForm.tsx b/web-client/src/views/StartCaseInternal/StatisticsForm.tsx
index f7209ee7e84..f9e1a099edf 100644
--- a/web-client/src/views/StartCaseInternal/StatisticsForm.tsx
+++ b/web-client/src/views/StartCaseInternal/StatisticsForm.tsx
@@ -49,7 +49,7 @@ export const StatisticsForm = connect(
           id={`deficiency-amount-${index}`}
           name={`statistics.${index}.irsDeficiencyAmount`}
           value={form.statistics[index].irsDeficiencyAmount || ''}
-          onBlur={validatePetitionFromPaperSequence}
+          onBlur={() => validatePetitionFromPaperSequence()}
           onValueChange={values => {
             updateStatisticsFormValueSequence({
               key: `statistics.${index}.irsDeficiencyAmount`,

From 479b89dc6a054158b051d2edf7b4bc007296444b Mon Sep 17 00:00:00 2001
From: Christopher Bisom <cbisom@flexion.us>
Date: Mon, 23 Sep 2024 09:09:38 -0400
Subject: [PATCH 25/33] 10074-bug-to-test: fix test

---
 .../journey/petitionsClerkUpdatesCaseDetail.ts                  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/web-client/integration-tests/journey/petitionsClerkUpdatesCaseDetail.ts b/web-client/integration-tests/journey/petitionsClerkUpdatesCaseDetail.ts
index f574dc4f233..df2968ca557 100644
--- a/web-client/integration-tests/journey/petitionsClerkUpdatesCaseDetail.ts
+++ b/web-client/integration-tests/journey/petitionsClerkUpdatesCaseDetail.ts
@@ -143,7 +143,7 @@ export const petitionsClerkUpdatesCaseDetail = cerebralTest => {
     });
     expect(cerebralTest.getState('alertError')).toEqual({
       messages: ['Select a case type', 'Select a case procedure'],
-      scrollToErrorNotification: false,
+      scrollToErrorNotification: true,
       title: 'Please correct the following errors on the page:',
     });
 

From c23a088841afeff4ea408f122c5ec4e03c4670f7 Mon Sep 17 00:00:00 2001
From: Christopher Bisom <cbisom@flexion.us>
Date: Mon, 23 Sep 2024 12:15:33 -0400
Subject: [PATCH 26/33] 10074-bug: update test name after review feedback

---
 .../presenter/actions/setValidationAlertErrorsAction.test.ts    | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/web-client/src/presenter/actions/setValidationAlertErrorsAction.test.ts b/web-client/src/presenter/actions/setValidationAlertErrorsAction.test.ts
index f4b05f4e5e3..1e79fd1ad78 100644
--- a/web-client/src/presenter/actions/setValidationAlertErrorsAction.test.ts
+++ b/web-client/src/presenter/actions/setValidationAlertErrorsAction.test.ts
@@ -142,7 +142,7 @@ describe('setValidationAlertErrors', () => {
     });
   });
 
-  it('state.alertError sets scrollToErrorNotification appropriately', async () => {
+  it('should set state.alertError to true when scrollToErrorNotification is true', async () => {
     const { state } = await runAction(setValidationAlertErrorsAction, {
       modules: {
         presenter,

From 1650c94f786c492d267cf03b0968666c38b260a8 Mon Sep 17 00:00:00 2001
From: Christopher Bisom <cbisom@flexion.us>
Date: Mon, 23 Sep 2024 12:28:39 -0400
Subject: [PATCH 27/33] 10074-bug: get openCompleteAndSendMessageModalSequence
 to mimic completeDocketEntryQCSequence

---
 .../sequences/openCompleteAndSendMessageModalSequence.ts      | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/web-client/src/presenter/sequences/openCompleteAndSendMessageModalSequence.ts b/web-client/src/presenter/sequences/openCompleteAndSendMessageModalSequence.ts
index 7e125543e4b..642c94d1a9c 100644
--- a/web-client/src/presenter/sequences/openCompleteAndSendMessageModalSequence.ts
+++ b/web-client/src/presenter/sequences/openCompleteAndSendMessageModalSequence.ts
@@ -10,6 +10,8 @@ import { setScrollToErrorNotificationAction } from '@web-client/presenter/action
 import { setShowModalFactoryAction } from '../actions/setShowModalFactoryAction';
 import { setValidationAlertErrorsAction } from '../actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
+import { startShowValidationAction } from '@web-client/presenter/actions/startShowValidationAction';
+import { stopShowValidationAction } from '@web-client/presenter/actions/stopShowValidationAction';
 import { updateMessageModalAfterQCAction } from '../actions/updateMessageModalAfterQCAction';
 import { validateDocketEntryAction } from '../actions/DocketEntry/validateDocketEntryAction';
 
@@ -19,6 +21,7 @@ export const openCompleteAndSendMessageModalSequence = [
   isWorkItemAlreadyCompletedAction,
   {
     no: [
+      startShowValidationAction,
       setFilersFromFilersMapAction,
       validateDocketEntryAction,
       {
@@ -28,6 +31,7 @@ export const openCompleteAndSendMessageModalSequence = [
           setValidationAlertErrorsAction,
         ],
         success: [
+          stopShowValidationAction,
           clearModalStateAction,
           refreshExternalDocumentTitleFromEventCodeAction,
           setPreviousDocumentDocketEntryAction,

From e85f96dc98dcf6340ee836d2f7119db6cb05e0b1 Mon Sep 17 00:00:00 2001
From: Christopher Bisom <cbisom@flexion.us>
Date: Mon, 23 Sep 2024 16:32:28 -0400
Subject: [PATCH 28/33] 10074-bug: update case advanced search form, which I
 overlooked

---
 .../presenter/sequences/submitCaseAdvancedSearchSequence.ts | 6 ++++--
 .../sequences/validateCaseAdvancedSearchFormSequence.ts     | 3 ++-
 2 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/web-client/src/presenter/sequences/submitCaseAdvancedSearchSequence.ts b/web-client/src/presenter/sequences/submitCaseAdvancedSearchSequence.ts
index 43eba100a1c..5327e9b9313 100644
--- a/web-client/src/presenter/sequences/submitCaseAdvancedSearchSequence.ts
+++ b/web-client/src/presenter/sequences/submitCaseAdvancedSearchSequence.ts
@@ -3,7 +3,8 @@ import { clearSearchResultsAction } from '../actions/AdvancedSearch/clearSearchR
 import { clearSearchTermAction } from '../actions/clearSearchTermAction';
 import { defaultCaseSearchDatesAction } from '../actions/AdvancedSearch/defaultCaseSearchDatesAction';
 import { setAdvancedSearchResultsAction } from '../actions/AdvancedSearch/setAdvancedSearchResultsAction';
-import { setAlertErrorAction } from '../actions/setAlertErrorAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
+import { setValidationAlertErrorsAction } from '@web-client/presenter/actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
 import { showProgressSequenceDecorator } from '../utilities/showProgressSequenceDecorator';
 import { startShowValidationAction } from '../actions/startShowValidationAction';
@@ -18,8 +19,9 @@ export const submitCaseAdvancedSearchSequence = [
   validateCaseAdvancedSearchAction,
   {
     error: [
-      setAlertErrorAction,
       setValidationErrorsAction,
+      setScrollToErrorNotificationAction,
+      setValidationAlertErrorsAction,
       clearSearchResultsAction,
     ],
     success: showProgressSequenceDecorator([
diff --git a/web-client/src/presenter/sequences/validateCaseAdvancedSearchFormSequence.ts b/web-client/src/presenter/sequences/validateCaseAdvancedSearchFormSequence.ts
index 8e254c02148..6dc992aba64 100644
--- a/web-client/src/presenter/sequences/validateCaseAdvancedSearchFormSequence.ts
+++ b/web-client/src/presenter/sequences/validateCaseAdvancedSearchFormSequence.ts
@@ -1,4 +1,5 @@
 import { clearAlertsAction } from '../actions/clearAlertsAction';
+import { setValidationAlertErrorsAction } from '@web-client/presenter/actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
 import { shouldValidateAction } from '../actions/shouldValidateAction';
 import { validateCaseAdvancedSearchAction } from '../actions/AdvancedSearch/validateCaseAdvancedSearchAction';
@@ -10,7 +11,7 @@ export const validateCaseAdvancedSearchFormSequence = [
     validate: [
       validateCaseAdvancedSearchAction,
       {
-        error: [setValidationErrorsAction],
+        error: [setValidationErrorsAction, setValidationAlertErrorsAction],
         success: [clearAlertsAction],
       },
     ],

From eebcfafb58cbf1fc8df68a9241af816037979c5c Mon Sep 17 00:00:00 2001
From: Christopher Bisom <cbisom@flexion.us>
Date: Mon, 23 Sep 2024 16:39:08 -0400
Subject: [PATCH 29/33] 10074-bug: update validateAddPetitionSequernce to sync
 ErrorNotification with live validation

---
 .../src/presenter/sequences/validateAddPetitionerSequence.ts   | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/web-client/src/presenter/sequences/validateAddPetitionerSequence.ts b/web-client/src/presenter/sequences/validateAddPetitionerSequence.ts
index 6dc3464fd69..f2fcfbe01c9 100644
--- a/web-client/src/presenter/sequences/validateAddPetitionerSequence.ts
+++ b/web-client/src/presenter/sequences/validateAddPetitionerSequence.ts
@@ -1,4 +1,5 @@
 import { clearAlertsAction } from '../actions/clearAlertsAction';
+import { setValidationAlertErrorsAction } from '@web-client/presenter/actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
 import { shouldValidateAction } from '../actions/shouldValidateAction';
 import { validateAddPetitionerAction } from '../actions/validateAddPetitionerAction';
@@ -10,7 +11,7 @@ export const validateAddPetitionerSequence = [
     validate: [
       validateAddPetitionerAction,
       {
-        error: [setValidationErrorsAction],
+        error: [setValidationErrorsAction, setValidationAlertErrorsAction],
         success: [clearAlertsAction],
       },
     ],

From 3d2e2fcd94a51b9b583bdf2532bebc9b0b0fe381 Mon Sep 17 00:00:00 2001
From: Christopher Bisom <cbisom@flexion.us>
Date: Mon, 23 Sep 2024 17:18:43 -0400
Subject: [PATCH 30/33] 10074-bug: update advanced search forms

---
 .../sequences/submitOpinionAdvancedSearchSequence.ts      | 8 +++++---
 .../sequences/submitOrderAdvancedSearchSequence.ts        | 8 +++++---
 .../submitPractitionerBarNumberSearchSequence.ts          | 6 ++++--
 .../sequences/submitPractitionerNameSearchSequence.ts     | 6 ++++--
 4 files changed, 18 insertions(+), 10 deletions(-)

diff --git a/web-client/src/presenter/sequences/submitOpinionAdvancedSearchSequence.ts b/web-client/src/presenter/sequences/submitOpinionAdvancedSearchSequence.ts
index c1252090886..24d8ca04d36 100644
--- a/web-client/src/presenter/sequences/submitOpinionAdvancedSearchSequence.ts
+++ b/web-client/src/presenter/sequences/submitOpinionAdvancedSearchSequence.ts
@@ -2,7 +2,8 @@ import { clearAlertsAction } from '../actions/clearAlertsAction';
 import { clearSearchResultsAction } from '../actions/AdvancedSearch/clearSearchResultsAction';
 import { clearSearchTermAction } from '../actions/clearSearchTermAction';
 import { setAdvancedSearchResultsAction } from '../actions/AdvancedSearch/setAdvancedSearchResultsAction';
-import { setAlertErrorAction } from '../actions/setAlertErrorAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
+import { setValidationAlertErrorsAction } from '@web-client/presenter/actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
 import { showProgressSequenceDecorator } from '../utilities/showProgressSequenceDecorator';
 import { startShowValidationAction } from '../actions/startShowValidationAction';
@@ -15,10 +16,11 @@ export const submitOpinionAdvancedSearchSequence =
     validateOpinionAdvancedSearchAction,
     {
       error: [
-        setAlertErrorAction,
+        startShowValidationAction,
         setValidationErrorsAction,
+        setScrollToErrorNotificationAction,
+        setValidationAlertErrorsAction,
         clearSearchResultsAction,
-        startShowValidationAction,
       ],
       success: showProgressSequenceDecorator([
         clearAlertsAction,
diff --git a/web-client/src/presenter/sequences/submitOrderAdvancedSearchSequence.ts b/web-client/src/presenter/sequences/submitOrderAdvancedSearchSequence.ts
index 2bfdb4ea0c6..cfb1bdfd238 100644
--- a/web-client/src/presenter/sequences/submitOrderAdvancedSearchSequence.ts
+++ b/web-client/src/presenter/sequences/submitOrderAdvancedSearchSequence.ts
@@ -2,7 +2,8 @@ import { clearAlertsAction } from '../actions/clearAlertsAction';
 import { clearSearchResultsAction } from '../actions/AdvancedSearch/clearSearchResultsAction';
 import { clearSearchTermAction } from '../actions/clearSearchTermAction';
 import { setAdvancedSearchResultsAction } from '../actions/AdvancedSearch/setAdvancedSearchResultsAction';
-import { setAlertErrorAction } from '../actions/setAlertErrorAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
+import { setValidationAlertErrorsAction } from '@web-client/presenter/actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
 import { showProgressSequenceDecorator } from '../utilities/showProgressSequenceDecorator';
 import { startShowValidationAction } from '../actions/startShowValidationAction';
@@ -14,10 +15,11 @@ export const submitOrderAdvancedSearchSequence = showProgressSequenceDecorator([
   validateOrderAdvancedSearchAction,
   {
     error: [
-      setAlertErrorAction,
+      startShowValidationAction,
       setValidationErrorsAction,
+      setScrollToErrorNotificationAction,
+      setValidationAlertErrorsAction,
       clearSearchResultsAction,
-      startShowValidationAction,
     ],
     success: [
       clearAlertsAction,
diff --git a/web-client/src/presenter/sequences/submitPractitionerBarNumberSearchSequence.ts b/web-client/src/presenter/sequences/submitPractitionerBarNumberSearchSequence.ts
index aaa214a1447..51e8178c50a 100644
--- a/web-client/src/presenter/sequences/submitPractitionerBarNumberSearchSequence.ts
+++ b/web-client/src/presenter/sequences/submitPractitionerBarNumberSearchSequence.ts
@@ -6,8 +6,9 @@ import { getPractitionerDetailAction } from '../actions/getPractitionerDetailAct
 import { hasPractitionerDetailAction } from '../actions/hasPractitionerDetailAction';
 import { navigateToPractitionerDetailSequence } from './navigateToPractitionerDetailSequence';
 import { setAdvancedSearchResultsAction } from '../actions/AdvancedSearch/setAdvancedSearchResultsAction';
-import { setAlertErrorAction } from '../actions/setAlertErrorAction';
 import { setPractitionerDetailAction } from '../actions/setPractitionerDetailAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
+import { setValidationAlertErrorsAction } from '@web-client/presenter/actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
 import { showProgressSequenceDecorator } from '../utilities/showProgressSequenceDecorator';
 import { startShowValidationAction } from '../actions/startShowValidationAction';
@@ -19,8 +20,9 @@ export const submitPractitionerBarNumberSearchSequence = [
   validatePractitionerSearchByBarNumberAction,
   {
     error: [
-      setAlertErrorAction,
       setValidationErrorsAction,
+      setScrollToErrorNotificationAction,
+      setValidationAlertErrorsAction,
       clearSearchResultsAction,
     ],
     success: showProgressSequenceDecorator([
diff --git a/web-client/src/presenter/sequences/submitPractitionerNameSearchSequence.ts b/web-client/src/presenter/sequences/submitPractitionerNameSearchSequence.ts
index 090c587f1b9..161d2e3adc4 100644
--- a/web-client/src/presenter/sequences/submitPractitionerNameSearchSequence.ts
+++ b/web-client/src/presenter/sequences/submitPractitionerNameSearchSequence.ts
@@ -2,7 +2,8 @@ import { clearAlertsAction } from '../actions/clearAlertsAction';
 import { clearSearchResultsAction } from '../actions/AdvancedSearch/clearSearchResultsAction';
 import { getPractitionersByNameAction } from '../actions/AdvancedSearch/getPractitionersByNameAction';
 import { setAdvancedSearchResultsAction } from '../actions/AdvancedSearch/setAdvancedSearchResultsAction';
-import { setAlertErrorAction } from '../actions/setAlertErrorAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
+import { setValidationAlertErrorsAction } from '@web-client/presenter/actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
 import { showProgressSequenceDecorator } from '../utilities/showProgressSequenceDecorator';
 import { startShowValidationAction } from '../actions/startShowValidationAction';
@@ -14,8 +15,9 @@ export const submitPractitionerNameSearchSequence = [
   validatePractitionerSearchByNameAction,
   {
     error: [
-      setAlertErrorAction,
       setValidationErrorsAction,
+      setScrollToErrorNotificationAction,
+      setValidationAlertErrorsAction,
       clearSearchResultsAction,
     ],
     success: showProgressSequenceDecorator([

From dd6d2194fca8c3ee9a09883158493245ed352128 Mon Sep 17 00:00:00 2001
From: Christopher Bisom <cbisom@flexion.us>
Date: Mon, 23 Sep 2024 17:38:54 -0400
Subject: [PATCH 31/33] 10074-bug: fix failing tests

---
 .../integration-tests/docketClerkOpinionAdvancedSearch.test.ts   | 1 +
 .../integration-tests/docketClerkOrderAdvancedSearch.test.ts     | 1 +
 2 files changed, 2 insertions(+)

diff --git a/web-client/integration-tests/docketClerkOpinionAdvancedSearch.test.ts b/web-client/integration-tests/docketClerkOpinionAdvancedSearch.test.ts
index 33422fd20e1..84eb9ecc53b 100644
--- a/web-client/integration-tests/docketClerkOpinionAdvancedSearch.test.ts
+++ b/web-client/integration-tests/docketClerkOpinionAdvancedSearch.test.ts
@@ -78,6 +78,7 @@ describe('Docket clerk opinion advanced search', () => {
 
     expect(cerebralTest.getState('alertError')).toEqual({
       messages: ['Start date cannot be in the future. Enter valid start date.'],
+      scrollToErrorNotification: true,
       title: 'Please correct the following errors:',
     });
 
diff --git a/web-client/integration-tests/docketClerkOrderAdvancedSearch.test.ts b/web-client/integration-tests/docketClerkOrderAdvancedSearch.test.ts
index 91087689bfe..ea5a875819c 100644
--- a/web-client/integration-tests/docketClerkOrderAdvancedSearch.test.ts
+++ b/web-client/integration-tests/docketClerkOrderAdvancedSearch.test.ts
@@ -155,6 +155,7 @@ describe('Docket clerk advanced order search', () => {
 
     expect(cerebralTest.getState('alertError')).toEqual({
       messages: ['Start date cannot be in the future. Enter valid start date.'],
+      scrollToErrorNotification: true,
       title: 'Please correct the following errors:',
     });
 

From 8fb1b8de5b18e786259c7944b8485f9128682e67 Mon Sep 17 00:00:00 2001
From: Christopher Bisom <cbisom@flexion.us>
Date: Tue, 24 Sep 2024 09:10:07 -0400
Subject: [PATCH 32/33] 10074-bug: update a few more overlooked forms

---
 .../src/presenter/actions/validateCaseDetailsAction.ts    | 1 +
 .../sequences/Login/submitChangePasswordSequence.ts       | 8 +-------
 .../Public/submitPublicCaseAdvancedSearchSequence.ts      | 8 +++++---
 .../Public/submitPublicCaseDocketNumberSearchSequence.ts  | 6 ++++--
 .../Public/submitPublicOpinionAdvancedSearchSequence.ts   | 8 +++++---
 .../Public/submitPublicOrderAdvancedSearchSequence.ts     | 8 +++++---
 .../sequences/submitCaseDocketNumberSearchSequence.ts     | 6 ++++--
 .../sequences/submitUpdatePractitionerUserSequence.ts     | 6 +++++-
 .../src/presenter/sequences/updateCaseDetailsSequence.ts  | 8 +++++++-
 9 files changed, 37 insertions(+), 22 deletions(-)

diff --git a/web-client/src/presenter/actions/validateCaseDetailsAction.ts b/web-client/src/presenter/actions/validateCaseDetailsAction.ts
index 8895fb56edb..0c587e79822 100644
--- a/web-client/src/presenter/actions/validateCaseDetailsAction.ts
+++ b/web-client/src/presenter/actions/validateCaseDetailsAction.ts
@@ -18,6 +18,7 @@ export const validateCaseDetailsAction = async ({
         caseDetail: {
           ...caseDetail,
           ...form,
+          caseType: form.caseType || null,
           petitionPaymentDate: form.petitionPaymentDate || null,
           petitionPaymentMethod: form.petitionPaymentMethod || null,
           preferredTrialCity: form.preferredTrialCity
diff --git a/web-client/src/presenter/sequences/Login/submitChangePasswordSequence.ts b/web-client/src/presenter/sequences/Login/submitChangePasswordSequence.ts
index 4e14b3c8eb5..45bcc1cfebb 100644
--- a/web-client/src/presenter/sequences/Login/submitChangePasswordSequence.ts
+++ b/web-client/src/presenter/sequences/Login/submitChangePasswordSequence.ts
@@ -5,11 +5,9 @@ import { navigateToLoginAction } from '@web-client/presenter/actions/Login/navig
 import { navigateToPathAction } from '@web-client/presenter/actions/navigateToPathAction';
 import { setAlertErrorAction } from '@web-client/presenter/actions/setAlertErrorAction';
 import { setSaveAlertsForNavigationAction } from '@web-client/presenter/actions/setSaveAlertsForNavigationAction';
-import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
 import { setTokenAction } from '@web-client/presenter/actions/Login/setTokenAction';
 import { setUserAction } from '@web-client/presenter/actions/setUserAction';
 import { setUserPermissionsAction } from '@web-client/presenter/actions/setUserPermissionsAction';
-import { setValidationAlertErrorsAction } from '@web-client/presenter/actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '@web-client/presenter/actions/setValidationErrorsAction';
 import { showProgressSequenceDecorator } from '../../utilities/showProgressSequenceDecorator';
 import { submitChangePasswordAction } from '@web-client/presenter/actions/Login/submitChangePasswordAction';
@@ -20,11 +18,7 @@ export const submitChangePasswordSequence = [
     clearAlertsAction,
     validateChangePasswordFormAction,
     {
-      error: [
-        setValidationErrorsAction,
-        setScrollToErrorNotificationAction,
-        setValidationAlertErrorsAction,
-      ],
+      error: [setValidationErrorsAction],
       success: [
         submitChangePasswordAction,
         {
diff --git a/web-client/src/presenter/sequences/Public/submitPublicCaseAdvancedSearchSequence.ts b/web-client/src/presenter/sequences/Public/submitPublicCaseAdvancedSearchSequence.ts
index a9f2cb6a30e..e2382cd2c0d 100644
--- a/web-client/src/presenter/sequences/Public/submitPublicCaseAdvancedSearchSequence.ts
+++ b/web-client/src/presenter/sequences/Public/submitPublicCaseAdvancedSearchSequence.ts
@@ -3,7 +3,8 @@ import { clearSearchResultsAction } from '../../actions/AdvancedSearch/clearSear
 import { clearSearchTermAction } from '../../actions/clearSearchTermAction';
 import { defaultCaseSearchDatesAction } from '@web-client/presenter/actions/AdvancedSearch/defaultCaseSearchDatesAction';
 import { setAdvancedSearchResultsAction } from '../../actions/AdvancedSearch/setAdvancedSearchResultsAction';
-import { setAlertErrorAction } from '../../actions/setAlertErrorAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
+import { setValidationAlertErrorsAction } from '@web-client/presenter/actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../../actions/setValidationErrorsAction';
 import { showProgressSequenceDecorator } from '../../utilities/showProgressSequenceDecorator';
 import { startShowValidationAction } from '@web-client/presenter/actions/startShowValidationAction';
@@ -16,10 +17,11 @@ export const submitPublicCaseAdvancedSearchSequence = [
   validateCaseAdvancedSearchAction,
   {
     error: [
-      setAlertErrorAction,
+      startShowValidationAction,
       setValidationErrorsAction,
+      setScrollToErrorNotificationAction,
+      setValidationAlertErrorsAction,
       clearSearchResultsAction,
-      startShowValidationAction,
     ],
     success: showProgressSequenceDecorator([
       clearAlertsAction,
diff --git a/web-client/src/presenter/sequences/Public/submitPublicCaseDocketNumberSearchSequence.ts b/web-client/src/presenter/sequences/Public/submitPublicCaseDocketNumberSearchSequence.ts
index 58260a841f8..7d725ba6b72 100644
--- a/web-client/src/presenter/sequences/Public/submitPublicCaseDocketNumberSearchSequence.ts
+++ b/web-client/src/presenter/sequences/Public/submitPublicCaseDocketNumberSearchSequence.ts
@@ -2,9 +2,10 @@ import { caseExistsAction } from '../../actions/caseExistsAction';
 import { clearSearchResultsAction } from '../../actions/AdvancedSearch/clearSearchResultsAction';
 import { clearSearchTermAction } from '../../actions/clearSearchTermAction';
 import { navigateToCaseDetailAction } from '../../actions/navigateToCaseDetailAction';
-import { setAlertErrorAction } from '../../actions/setAlertErrorAction';
 import { setDocketNumberFromAdvancedSearchAction } from '../../actions/AdvancedSearch/setDocketNumberFromAdvancedSearchAction';
 import { setNoMatchesCaseSearchAction } from '../../actions/AdvancedSearch/setNoMatchesCaseSearchAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
+import { setValidationAlertErrorsAction } from '@web-client/presenter/actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../../actions/setValidationErrorsAction';
 import { showProgressSequenceDecorator } from '../../utilities/showProgressSequenceDecorator';
 import { startShowValidationAction } from '../../actions/startShowValidationAction';
@@ -17,8 +18,9 @@ export const submitPublicCaseDocketNumberSearchSequence = [
   validateCaseDocketNumberSearchAction,
   {
     error: [
-      setAlertErrorAction,
       setValidationErrorsAction,
+      setScrollToErrorNotificationAction,
+      setValidationAlertErrorsAction,
       clearSearchResultsAction,
     ],
     success: showProgressSequenceDecorator([
diff --git a/web-client/src/presenter/sequences/Public/submitPublicOpinionAdvancedSearchSequence.ts b/web-client/src/presenter/sequences/Public/submitPublicOpinionAdvancedSearchSequence.ts
index 373f8b0b565..54abfff661f 100644
--- a/web-client/src/presenter/sequences/Public/submitPublicOpinionAdvancedSearchSequence.ts
+++ b/web-client/src/presenter/sequences/Public/submitPublicOpinionAdvancedSearchSequence.ts
@@ -2,7 +2,8 @@ import { clearAlertsAction } from '../../actions/clearAlertsAction';
 import { clearSearchResultsAction } from '../../actions/AdvancedSearch/clearSearchResultsAction';
 import { clearSearchTermAction } from '../../actions/clearSearchTermAction';
 import { setAdvancedSearchResultsAction } from '../../actions/AdvancedSearch/setAdvancedSearchResultsAction';
-import { setAlertErrorAction } from '../../actions/setAlertErrorAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
+import { setValidationAlertErrorsAction } from '@web-client/presenter/actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../../actions/setValidationErrorsAction';
 import { showProgressSequenceDecorator } from '../../utilities/showProgressSequenceDecorator';
 import { startShowValidationAction } from '../../actions/startShowValidationAction';
@@ -14,10 +15,11 @@ export const submitPublicOpinionAdvancedSearchSequence = [
   validateOpinionAdvancedSearchAction,
   {
     error: [
-      setAlertErrorAction,
+      startShowValidationAction,
       setValidationErrorsAction,
+      setScrollToErrorNotificationAction,
+      setValidationAlertErrorsAction,
       clearSearchResultsAction,
-      startShowValidationAction,
     ],
     success: showProgressSequenceDecorator([
       clearAlertsAction,
diff --git a/web-client/src/presenter/sequences/Public/submitPublicOrderAdvancedSearchSequence.ts b/web-client/src/presenter/sequences/Public/submitPublicOrderAdvancedSearchSequence.ts
index 63bfa24cab7..d4499450b7d 100644
--- a/web-client/src/presenter/sequences/Public/submitPublicOrderAdvancedSearchSequence.ts
+++ b/web-client/src/presenter/sequences/Public/submitPublicOrderAdvancedSearchSequence.ts
@@ -2,7 +2,8 @@ import { clearAlertsAction } from '../../actions/clearAlertsAction';
 import { clearSearchResultsAction } from '../../actions/AdvancedSearch/clearSearchResultsAction';
 import { clearSearchTermAction } from '../../actions/clearSearchTermAction';
 import { setAdvancedSearchResultsAction } from '../../actions/AdvancedSearch/setAdvancedSearchResultsAction';
-import { setAlertErrorAction } from '../../actions/setAlertErrorAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
+import { setValidationAlertErrorsAction } from '@web-client/presenter/actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../../actions/setValidationErrorsAction';
 import { showProgressSequenceDecorator } from '../../utilities/showProgressSequenceDecorator';
 import { startShowValidationAction } from '../../actions/startShowValidationAction';
@@ -14,10 +15,11 @@ export const submitPublicOrderAdvancedSearchSequence = [
   validateOrderAdvancedSearchAction,
   {
     error: [
-      setAlertErrorAction,
+      startShowValidationAction,
       setValidationErrorsAction,
+      setScrollToErrorNotificationAction,
+      setValidationAlertErrorsAction,
       clearSearchResultsAction,
-      startShowValidationAction,
     ],
     success: showProgressSequenceDecorator([
       clearAlertsAction,
diff --git a/web-client/src/presenter/sequences/submitCaseDocketNumberSearchSequence.ts b/web-client/src/presenter/sequences/submitCaseDocketNumberSearchSequence.ts
index 344c361af12..e7a74a3b50a 100644
--- a/web-client/src/presenter/sequences/submitCaseDocketNumberSearchSequence.ts
+++ b/web-client/src/presenter/sequences/submitCaseDocketNumberSearchSequence.ts
@@ -2,9 +2,10 @@ import { caseExistsAction } from '../actions/caseExistsAction';
 import { clearSearchResultsAction } from '../actions/AdvancedSearch/clearSearchResultsAction';
 import { clearSearchTermAction } from '../actions/clearSearchTermAction';
 import { navigateToCaseDetailAction } from '../actions/navigateToCaseDetailAction';
-import { setAlertErrorAction } from '../actions/setAlertErrorAction';
 import { setDocketNumberFromAdvancedSearchAction } from '../actions/AdvancedSearch/setDocketNumberFromAdvancedSearchAction';
 import { setNoMatchesCaseSearchAction } from '../actions/AdvancedSearch/setNoMatchesCaseSearchAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
+import { setValidationAlertErrorsAction } from '@web-client/presenter/actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
 import { showProgressSequenceDecorator } from '../utilities/showProgressSequenceDecorator';
 import { startShowValidationAction } from '../actions/startShowValidationAction';
@@ -17,8 +18,9 @@ export const submitCaseDocketNumberSearchSequence = [
   validateCaseDocketNumberSearchAction,
   {
     error: [
-      setAlertErrorAction,
       setValidationErrorsAction,
+      setScrollToErrorNotificationAction,
+      setValidationAlertErrorsAction,
       clearSearchResultsAction,
     ],
     success: showProgressSequenceDecorator([
diff --git a/web-client/src/presenter/sequences/submitUpdatePractitionerUserSequence.ts b/web-client/src/presenter/sequences/submitUpdatePractitionerUserSequence.ts
index 947429eaa5f..d8c300bc0e3 100644
--- a/web-client/src/presenter/sequences/submitUpdatePractitionerUserSequence.ts
+++ b/web-client/src/presenter/sequences/submitUpdatePractitionerUserSequence.ts
@@ -26,7 +26,11 @@ export const submitUpdatePractitionerUserSequence = [
   startShowValidationAction,
   validatePractitionerAction,
   {
-    error: [setValidationErrorsAction, setValidationAlertErrorsAction],
+    error: [
+      setValidationErrorsAction,
+      setScrollToErrorNotificationAction,
+      setValidationAlertErrorsAction,
+    ],
     success: [
       setWaitingForResponseAction,
       hasUpdatedEmailFactoryAction('updatedEmail'),
diff --git a/web-client/src/presenter/sequences/updateCaseDetailsSequence.ts b/web-client/src/presenter/sequences/updateCaseDetailsSequence.ts
index e2771fcd1cc..bda593a21bb 100644
--- a/web-client/src/presenter/sequences/updateCaseDetailsSequence.ts
+++ b/web-client/src/presenter/sequences/updateCaseDetailsSequence.ts
@@ -6,6 +6,8 @@ import { setCaseAction } from '../actions/setCaseAction';
 import { setCaseDetailShowEditPetitionFalseAction } from '../actions/setCaseDetailShowEditPetitionFalseAction';
 import { setCaseTypeAction } from '../actions/setCaseTypeAction';
 import { setSaveAlertsForNavigationAction } from '../actions/setSaveAlertsForNavigationAction';
+import { setScrollToErrorNotificationAction } from '@web-client/presenter/actions/setScrollToErrorNotificationAction';
+import { setValidationAlertErrorsAction } from '@web-client/presenter/actions/setValidationAlertErrorsAction';
 import { setValidationErrorsAction } from '../actions/setValidationErrorsAction';
 import { setupCurrentPageAction } from '../actions/setupCurrentPageAction';
 import { startShowValidationAction } from '../actions/startShowValidationAction';
@@ -17,7 +19,11 @@ export const updateCaseDetailsSequence = [
   startShowValidationAction,
   validateCaseDetailsAction,
   {
-    error: [setAlertErrorAction, setValidationErrorsAction],
+    error: [
+      setValidationErrorsAction,
+      setScrollToErrorNotificationAction,
+      setValidationAlertErrorsAction,
+    ],
     success: [
       setupCurrentPageAction('Interstitial'),
       setCaseTypeAction,

From 8a08e965361d752f27ad298d5363ab6b09e80fee Mon Sep 17 00:00:00 2001
From: Christopher Bisom <cbisom@flexion.us>
Date: Tue, 24 Sep 2024 10:35:50 -0400
Subject: [PATCH 33/33] 10074-bug: fix failing tests

---
 .../integration-tests/docketClerkOpinionAdvancedSearch.test.ts | 2 +-
 .../integration-tests/docketClerkOrderAdvancedSearch.test.ts   | 3 ++-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/web-client/integration-tests/docketClerkOpinionAdvancedSearch.test.ts b/web-client/integration-tests/docketClerkOpinionAdvancedSearch.test.ts
index 84eb9ecc53b..45f5d86c174 100644
--- a/web-client/integration-tests/docketClerkOpinionAdvancedSearch.test.ts
+++ b/web-client/integration-tests/docketClerkOpinionAdvancedSearch.test.ts
@@ -79,7 +79,7 @@ describe('Docket clerk opinion advanced search', () => {
     expect(cerebralTest.getState('alertError')).toEqual({
       messages: ['Start date cannot be in the future. Enter valid start date.'],
       scrollToErrorNotification: true,
-      title: 'Please correct the following errors:',
+      title: 'Please correct the following errors on the page:',
     });
 
     await cerebralTest.runSequence('advancedSearchTabChangeSequence');
diff --git a/web-client/integration-tests/docketClerkOrderAdvancedSearch.test.ts b/web-client/integration-tests/docketClerkOrderAdvancedSearch.test.ts
index ea5a875819c..b52db3c644b 100644
--- a/web-client/integration-tests/docketClerkOrderAdvancedSearch.test.ts
+++ b/web-client/integration-tests/docketClerkOrderAdvancedSearch.test.ts
@@ -147,6 +147,7 @@ describe('Docket clerk advanced order search', () => {
     cerebralTest.setState('advancedSearchForm', {
       orderSearch: {
         dateRange: DATE_RANGE_SEARCH_OPTIONS.CUSTOM_DATES,
+        scrollToErrorNotification: true,
         startDate: '01/01/3001',
       },
     });
@@ -156,7 +157,7 @@ describe('Docket clerk advanced order search', () => {
     expect(cerebralTest.getState('alertError')).toEqual({
       messages: ['Start date cannot be in the future. Enter valid start date.'],
       scrollToErrorNotification: true,
-      title: 'Please correct the following errors:',
+      title: 'Please correct the following errors on the page:',
     });
 
     await cerebralTest.runSequence('advancedSearchTabChangeSequence');