fix: fixed duplicate event firing on post SSO login registration (#1381)

openedx · Nov 27, 2024 · cd20039 · cd20039
1 parent 7339aec
commit cd20039
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/src/login/LoginPage.jsx b/src/login/LoginPage.jsx
@@ -160,7 +160,10 @@ const LoginPage = (props) => {
       webElementText: ELEMENT_TEXT.SIGN_IN,
       webElementName: ELEMENT_NAME.SIGN_IN,
     };
-    dispatch(setCohesionEventStates(eventData));
+    if (currentProvider) {
+      dispatch(setCohesionEventStates(eventData));
+    }
+
     if (showResetPasswordSuccessBanner) {
       props.dismissPasswordResetBanner();
     }

diff --git a/src/register/RegistrationPage.jsx b/src/register/RegistrationPage.jsx
@@ -278,7 +278,9 @@ const RegistrationPage = (props) => {
       webElementText: ELEMENT_TEXT.CREATE_ACCOUNT,
       webElementName: ELEMENT_NAME.CREATE_ACCOUNT,
     };
-    dispatch(setCohesionEventStates(eventData));
+    if (currentProvider) {
+      dispatch(setCohesionEventStates(eventData));
+    }
     registerUser();
   };