Merge pull request #1801 from GSA/1786-fix-bf-change-event

PXBF-1786-fix-bf-change-event: async data handling in modal
GSA · Sep 13, 2024 · 779ece4 · 779ece4
2 parents 1d7bc1d + c6acc5a
commit 779ece4
Showing 1 changed file with 26 additions and 20 deletions.
diff --git a/benefit-finder/src/shared/components/Modal/index.jsx b/benefit-finder/src/shared/components/Modal/index.jsx
@@ -116,27 +116,33 @@ const Modal = ({
 
   // handle dataLayer
   useEffect(() => {
-    modalOpen === true &&
-      dataLayerUtils.dataLayerPush(window, {
-        event: modal.event,
-        bfData: {
-          pageView: modal.bfData.pageView,
-          viewTitle: `${dataLayerValue.viewTitle} modal`,
-        },
-      })
-    // handle dataLayer
-    modalOpen === true &&
-      dataLayerUtils.dataLayerPush(window, {
-        event: errors.event,
-        bfData: {
-          errors: '',
-          errorCount: {
-            number: 0,
-            string: `0`,
+    const handleModalData = async () => {
+      modalOpen === true &&
+        dataLayerUtils.dataLayerPush(window, {
+          event: modal.event,
+          bfData: {
+            pageView: modal.bfData.pageView,
+            viewTitle: `${dataLayerValue.viewTitle} modal`,
           },
-          formSuccess: true,
-        },
-      })
+        })
+    }
+
+    // async so we can handle duplicates if needed
+    handleModalData().then(() => {
+      // handle dataLayer
+      modalOpen === true &&
+        dataLayerUtils.dataLayerPush(window, {
+          event: errors.event,
+          bfData: {
+            errors: '',
+            errorCount: {
+              number: 0,
+              string: `0`,
+            },
+            formSuccess: true,
+          },
+        })
+    })
   }, [])
 
   /**