feat(sign-up): consent request management

ref: MANAGER-11426 Signed-off-by: Jacques Larique <[email protected]>
ovh · Jul 27, 2023 · 3cd8934 · 3cd8934
1 parent 4c905b1
commit 3cd8934
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 20 deletions.
diff --git a/packages/manager/apps/sign-up/src/routing.js b/packages/manager/apps/sign-up/src/routing.js
@@ -76,20 +76,19 @@ export const state = {
       me,
       signUp,
     ) => (smsConsent) =>
-      signUp.saveNic(me.model).then(() => {
-        if (smsConsent) {
-          signUp.giveSmsConsent();
-        }
-        // for tracking purposes
-        if ($window.sessionStorage) {
-          $window.sessionStorage.setItem('ovhSessionSuccess', true);
-        }
-        // manage redirection
-        const redirectUrl = getRedirectLocation(me.nichandle);
-        if (redirectUrl) {
-          $window.location.assign(redirectUrl);
-        }
-      }),
+      signUp.saveNic(me.model).then(() =>
+        signUp.sendSmsConsent(smsConsent).then(() => {
+          // for tracking purposes
+          if ($window.sessionStorage) {
+            $window.sessionStorage.setItem('ovhSessionSuccess', true);
+          }
+          // manage redirection
+          const redirectUrl = getRedirectLocation(me.nichandle);
+          if (redirectUrl) {
+            $window.location.assign(redirectUrl);
+          }
+        }),
+      ),
 
     steps: () => [
       {

diff --git a/packages/manager/modules/sign-up/src/service.js b/packages/manager/modules/sign-up/src/service.js
@@ -40,14 +40,14 @@ export default class SignUpService {
     return this.$http.put('/me', nicInfos);
   }
 
-  giveSmsConsent() {
+  sendSmsConsent(consent = false) {
     return this.$http.put('/me/marketing', {
-      denyAll: false,
+      denyAll: consent !== true,
       sms: {
-        events: true,
-        newProductRecommendation: true,
-        newsletter: true,
-        offerAndDiscount: true,
+        events: consent,
+        newProductRecommendation: consent,
+        newsletter: consent,
+        offerAndDiscount: consent,
       },
     });
   }