Remove redundant OTP events (#7542)

Automattic · Oct 26, 2023 · 34c0175 · 34c0175
1 parent 641fb60
commit 34c0175
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 44 deletions.
diff --git a/changelog/fix-remove-redundant-tracks-events b/changelog/fix-remove-redundant-tracks-events
@@ -0,0 +1,4 @@
+Significance: patch
+Type: dev
+
+Remove redundant Tracks events in WooPay OTP flow
diff --git a/client/checkout/woopay/email-input-iframe.js b/client/checkout/woopay/email-input-iframe.js
@@ -215,11 +215,6 @@ export const handleWooPayEmailInput = async (
 		window.addEventListener( 'resize', setPopoverPosition );
 
 		iframe.classList.add( 'open' );
-		wcpayTracks.recordUserEvent(
-			wcpayTracks.events.WOOPAY_OTP_START,
-			[],
-			true
-		);
 	} );
 
 	// Add the iframe and iframe arrow to the wrapper.
@@ -539,11 +534,6 @@ export const handleWooPayEmailInput = async (
 				closeLoginSessionIframe();
 				break;
 			case 'redirect_to_woopay_skip_session_init':
-				wcpayTracks.recordUserEvent(
-					wcpayTracks.events.WOOPAY_OTP_COMPLETE,
-					[],
-					true
-				);
 				if ( e.data.redirectUrl ) {
 					window.location = appendRedirectionParams(
 						e.data.redirectUrl
@@ -552,11 +542,6 @@ export const handleWooPayEmailInput = async (
 				break;
 			case 'redirect_to_platform_checkout':
 			case 'redirect_to_woopay':
-				wcpayTracks.recordUserEvent(
-					wcpayTracks.events.WOOPAY_OTP_COMPLETE,
-					[],
-					true
-				);
 				api.initWooPay(
 					woopayEmailInput.value,
 					e.data.platformCheckoutUserSession
@@ -581,11 +566,6 @@ export const handleWooPayEmailInput = async (
 					} );
 				break;
 			case 'otp_validation_failed':
-				wcpayTracks.recordUserEvent(
-					wcpayTracks.events.WOOPAY_OTP_FAILED,
-					[],
-					true
-				);
 				break;
 			case 'close_modal':
 				closeIframe();

diff --git a/client/checkout/woopay/express-button/express-checkout-iframe.js b/client/checkout/woopay/express-button/express-checkout-iframe.js
@@ -15,7 +15,6 @@ import {
 	validateEmail,
 	appendRedirectionParams,
 } from '../utils';
-import wcpayTracks from 'tracks';
 
 export const expressCheckoutIframe = async ( api, context, emailSelector ) => {
 	const woopayEmailInput = await getTargetElement( emailSelector );
@@ -126,11 +125,6 @@ export const expressCheckoutIframe = async ( api, context, emailSelector ) => {
 		window.addEventListener( 'resize', setPopoverPosition );
 
 		iframe.classList.add( 'open' );
-		wcpayTracks.recordUserEvent(
-			wcpayTracks.events.WOOPAY_OTP_START,
-			[],
-			true
-		);
 	} );
 
 	// Add the iframe to the wrapper.
@@ -212,11 +206,6 @@ export const expressCheckoutIframe = async ( api, context, emailSelector ) => {
 				userEmail = e.data.userEmail;
 				break;
 			case 'redirect_to_woopay_skip_session_init':
-				wcpayTracks.recordUserEvent(
-					wcpayTracks.events.WOOPAY_OTP_COMPLETE,
-					[],
-					true
-				);
 				if ( e.data.redirectUrl ) {
 					window.location = appendRedirectionParams(
 						e.data.redirectUrl
@@ -225,11 +214,6 @@ export const expressCheckoutIframe = async ( api, context, emailSelector ) => {
 				break;
 			case 'redirect_to_platform_checkout':
 			case 'redirect_to_woopay':
-				wcpayTracks.recordUserEvent(
-					wcpayTracks.events.WOOPAY_OTP_COMPLETE,
-					[],
-					true
-				);
 				api.initWooPay(
 					userEmail || e.data.userEmail,
 					e.data.platformCheckoutUserSession
@@ -254,11 +238,6 @@ export const expressCheckoutIframe = async ( api, context, emailSelector ) => {
 				} );
 				break;
 			case 'otp_validation_failed':
-				wcpayTracks.recordUserEvent(
-					wcpayTracks.events.WOOPAY_OTP_FAILED,
-					[],
-					true
-				);
 				break;
 			case 'close_modal':
 				closeIframe();

diff --git a/client/tracks/index.js b/client/tracks/index.js
@@ -108,9 +108,6 @@ const events = {
 	TRANSACTIONS_DOWNLOAD_CSV_CLICK: 'wcpay_transactions_download_csv_click',
 	WOOPAY_EMAIL_CHECK: 'checkout_email_address_woopay_check',
 	WOOPAY_OFFERED: 'woopay_offered',
-	WOOPAY_OTP_START: 'woopay_otp_prompt_start',
-	WOOPAY_OTP_COMPLETE: 'woopay_otp_prompt_complete',
-	WOOPAY_OTP_FAILED: 'woopay_otp_prompt_failed',
 	WOOPAY_AUTO_REDIRECT: 'checkout_woopay_auto_redirect',
 	WOOPAY_SKIPPED: 'woopay_skipped',
 	WOOPAY_BUTTON_LOAD: 'woopay_button_load',