feat: final changes

edx · Jul 31, 2024 · df39590 · df39590
1 parent 32d0d63
commit df39590
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 4 deletions.
diff --git a/src/common-ui/SocialAuthButtons/index.jsx b/src/common-ui/SocialAuthButtons/index.jsx
@@ -38,7 +38,7 @@ export const SocialAuthButton = forwardRef(({
     // ...registrationFields,
     totalRegistrationTime: 0,
     app_name: 'onboarding-component',
-    marketing_emails_opt_in: true,
+    marketing_emails_opt_in: false,
   };
 
   if (!provider) {

diff --git a/src/forms/common-components/AuthenticatedRedirection.jsx b/src/forms/common-components/AuthenticatedRedirection.jsx
@@ -37,7 +37,6 @@ const AuthenticatedRedirection = ({
 
   if (success) {
     let finalRedirectUrl = '';
-    debugger;
     // If we're in a third party auth pipeline, we must complete the pipeline
     // once user has successfully logged in. Otherwise, redirect to the specified redirect url.
     // Note: For multiple enterprise use case, we need to make sure that user first visits the
@@ -59,9 +58,12 @@ const AuthenticatedRedirection = ({
     }
 
     if (isLinkTracked) {
+      console.log('oct1 before redirect', { finalRedirectUrl });
       setTimeout(() => { window.location.href = finalRedirectUrl; }, LINK_TIMEOUT);
+    } else {
+      console.log('oct2 before redirect', { finalRedirectUrl });
+      window.location.href = finalRedirectUrl;
     }
-    window.location.href = finalRedirectUrl;
   }
 
   return null;

diff --git a/src/forms/progressive-profiling-popup/index.jsx b/src/forms/progressive-profiling-popup/index.jsx
@@ -254,7 +254,7 @@ const ProgressiveProfilingForm = () => {
         redirectUrl={redirectUrl}
         finishAuthUrl={finishAuthUrl}
         isLinkTracked
-        shouldUseRedirectUrl={isRedirectedFromSSOPipeline}
+        // shouldUseRedirectUrl={isRedirectedFromSSOPipeline}
       />
       <h1
         className="display-1 font-italic text-center mb-4"