fix(analytics): adding skip verify email

blockchain · Jul 13, 2021 · be0512c · be0512c
1 parent e061499
commit be0512c
Show file tree

Hide file tree

Showing 5 changed files with 45 additions and 9 deletions.
diff --git a/packages/blockchain-wallet-v4-frontend/src/data/modules/securityCenter/actionTypes.ts b/packages/blockchain-wallet-v4-frontend/src/data/modules/securityCenter/actionTypes.ts
@@ -6,6 +6,8 @@ export const VERIFY_EMAIL_CODE = '@COMPONENT.VERIFY_EMAIL_CODE'
 
 export const RESEND_VERIFY_EMAIL = '@COMPONENT.RESEND_VERIFY_EMAIL'
 
+export const SKIP_VERIFY_EMAIL = '@COMPONENT.SKIP_VERIFY_EMAIL'
+
 export const SEND_CONFIRMATION_CODE_EMAIL = '@COMPONENT.SEND_CONFIRMATION_CODE_EMAIL'
 
 export const GET_GOOGLE_AUTHENTICATOR_SECRET_URL = '@COMPONENT.GET_GOOGLE_AUTHENTICATOR_SECRET_URL'

diff --git a/packages/blockchain-wallet-v4-frontend/src/data/modules/securityCenter/actions.js b/packages/blockchain-wallet-v4-frontend/src/data/modules/securityCenter/actions.js
@@ -25,6 +25,11 @@ export const resendVerifyEmail = (email) => ({
   type: AT.RESEND_VERIFY_EMAIL
 })
 
+export const skipVerifyEmail = (email) => ({
+  payload: { email },
+  type: AT.SKIP_VERIFY_EMAIL
+})
+
 export const getGoogleAuthenticatorSecretUrl = () => ({
   type: AT.GET_GOOGLE_AUTHENTICATOR_SECRET_URL
 })

diff --git a/packages/blockchain-wallet-v4-frontend/src/middleware/analyticsMiddleware/index.ts b/packages/blockchain-wallet-v4-frontend/src/middleware/analyticsMiddleware/index.ts
@@ -578,10 +578,9 @@ const analyticsMiddleware = () => (store) => (next) => (action) => {
         const email = state.profile.userData.getOrElse({})?.email
         const tier = state.profile.userData.getOrElse({})?.tiers.current
         const id = state.walletPath.wallet.guid
-
         const origin = 'SIGN_UP'
 
-        analytics.push(AnalyticsKey.AMOUNT_SWITCHED, {
+        analytics.push(AnalyticsKey.EMAIL_VERIFICATION_REQUESTED, {
           properties: {
             id,
             origin,
@@ -602,10 +601,32 @@ const analyticsMiddleware = () => (store) => (next) => (action) => {
         const email = state.profile.userData.getOrElse({})?.email
         const tier = state.profile.userData.getOrElse({})?.tiers.current
         const id = state.walletPath.wallet.guid
-
         const origin = 'VERIFICATION'
 
-        analytics.push(AnalyticsKey.AMOUNT_SWITCHED, {
+        analytics.push(AnalyticsKey.EMAIL_VERIFICATION_REQUESTED, {
+          properties: {
+            id,
+            origin,
+            originalTimestamp: getOriginalTimestamp()
+          },
+          traits: {
+            email,
+            nabuId,
+            tier
+          }
+        })
+
+        break
+      }
+      case AT.modules.securityCenter.SKIP_VERIFY_EMAIL: {
+        const state = store.getState()
+        const nabuId = state.profile.userData.getOrElse({})?.id
+        const email = state.profile.userData.getOrElse({})?.email
+        const tier = state.profile.userData.getOrElse({})?.tiers.current
+        const id = state.walletPath.wallet.guid
+        const origin = 'SIGN_UP'
+
+        analytics.push(AnalyticsKey.EMAIL_VERIFICATION_SKIPPED, {
           properties: {
             id,
             origin,

diff --git a/packages/blockchain-wallet-v4-frontend/src/middleware/analyticsMiddleware/types.ts b/packages/blockchain-wallet-v4-frontend/src/middleware/analyticsMiddleware/types.ts
@@ -17,6 +17,7 @@ enum AnalyticsKey {
   DEPOSIT_METHOD_SELECTED = 'Deposit Method Selected',
   DEPOSIT_VIEWED = 'Deposit Viewed',
   EMAIL_VERIFICATION_REQUESTED = 'Email Verification Requested',
+  EMAIL_VERIFICATION_SKIPPED = 'Email Verification Skipped',
   IMPORT_ADDRESS_CLICKED = 'Import Address Clicked',
   INTEREST_CLICKED = 'Interest Clicked',
   INTEREST_DEPOSIT_AMOUNT_ENTERED = 'Interest Deposit Amount Entered',
@@ -239,10 +240,14 @@ type DepositMethodSelectedPayload = BasePayload & {
 
 type DepositViewedPayload = BasePayload & PageViewPayload & {}
 
-type EmailVerificationClickedOrigin = 'SIGN_UP' | 'VERIFICATION'
+type EmailVerificationSkippedPayload = BasePayload & {
+  origin: 'SIGN_UP'
+}
+
+type EmailVerificationRequestedOrigin = 'SIGN_UP' | 'VERIFICATION'
 
-type EmailVerificationClickedPayload = BasePayload & {
-  origin: EmailVerificationClickedOrigin
+type EmailVerificationRequestedPayload = BasePayload & {
+  origin: EmailVerificationRequestedOrigin
 }
 
 type ImportAddressClickedPayload = BasePayload & {}
@@ -571,7 +576,8 @@ type AnalyticsProperties =
   | DepositClickedPayload
   | DepositMethodSelectedPayload
   | DepositViewedPayload
-  | EmailVerificationClickedPayload
+  | EmailVerificationRequestedPayload
+  | EmailVerificationSkippedPayload
   | ImportAddressClickedPayload
   | InterestClickedPayload
   | InterestDepositAmountEnteredPayload
@@ -637,7 +643,7 @@ export type {
   BuySellClickedOrigin,
   DashboardClickedOrigin,
   DepositClickedOrigin,
-  EmailVerificationClickedOrigin,
+  EmailVerificationRequestedOrigin,
   InterestDepositClickedOrigin,
   InterestSubmitInformationClickedOrigin,
   InterestWithdrawalClickedOrigin,

diff --git a/packages/blockchain-wallet-v4-frontend/src/scenes/VerifyEmail/index.tsx b/packages/blockchain-wallet-v4-frontend/src/scenes/VerifyEmail/index.tsx
@@ -27,8 +27,10 @@ class VerifyEmailContainer extends React.PureComponent<Props> {
   }
 
   skipVerification = () => {
+    const { email } = this.props
     this.props.authActions.setRegisterEmail(undefined)
     this.props.analyticsActions.logEvent(DISMISS_VERIFICATION)
+    this.props.securityCenterActions.skipVerifyEmail(email)
     this.props.routerActions.push('/home')
     // for first time login users we need to run goal since this is a first page we show them
     this.props.saveGoal('welcomeModal', { firstLogin: true })