فهرست منبع

Merge pull request #32096 from overleaf/rh-rm-cio-integration-segmentation

Remove customerio-integration segmentation from all events

GitOrigin-RevId: 628772f61f2ace0fc6b6f4ddbabbab94057188ca
roo hutton 5 ماه پیش
والد
کامیت
15714582e4

+ 0 - 11
services/web/app/src/Features/Subscription/RecurlyEventHandler.mjs

@@ -13,9 +13,6 @@ async function sendRecurlyAnalyticsEvent(event, eventData) {
   if (!ObjectId.isValid(userId)) {
     return
   }
-
-  eventData['customerio-integration'] = true
-
   const subscription =
     await SubscriptionLocator.promises.getUsersSubscription(userId)
 
@@ -82,7 +79,6 @@ async function _sendSubscriptionResumedEvent(userId, eventData) {
       plan_code: planCode,
       subscriptionId,
       payment_provider: 'recurly',
-      'customerio-integration': eventData['customerio-integration'],
     }
   )
   AnalyticsManager.setUserPropertyForUserInBackground(
@@ -105,7 +101,6 @@ async function _sendSubscriptionPausedEvent(userId, eventData) {
       plan_code: planCode,
       subscriptionId,
       payment_provider: 'recurly',
-      'customerio-integration': eventData['customerio-integration'],
     }
   )
   AnalyticsManager.setUserPropertyForUserInBackground(
@@ -128,7 +123,6 @@ async function _sendSubscriptionStartedEvent(userId, eventData) {
       has_ai_add_on: hasAiAddOn,
       subscriptionId,
       payment_provider: 'recurly',
-      'customerio-integration': eventData['customerio-integration'],
     }
   )
   AnalyticsManager.setUserPropertyForUserInBackground(
@@ -165,7 +159,6 @@ async function _sendSubscriptionUpdatedEvent(userId, eventData) {
       has_ai_add_on: hasAiAddOn,
       subscriptionId,
       payment_provider: 'recurly',
-      'customerio-integration': eventData['customerio-integration'],
     }
   )
   AnalyticsManager.setUserPropertyForUserInBackground(
@@ -198,7 +191,6 @@ async function _sendSubscriptionCancelledEvent(userId, eventData) {
       has_ai_add_on: hasAiAddOn,
       subscriptionId,
       payment_provider: 'recurly',
-      'customerio-integration': eventData['customerio-integration'],
     }
   )
   AnalyticsManager.setUserPropertyForUserInBackground(
@@ -226,7 +218,6 @@ async function _sendSubscriptionExpiredEvent(userId, eventData) {
       has_ai_add_on: hasAiAddOn,
       subscriptionId,
       payment_provider: 'recurly',
-      'customerio-integration': eventData['customerio-integration'],
     }
   )
   AnalyticsManager.setUserPropertyForUserInBackground(
@@ -259,7 +250,6 @@ async function _sendSubscriptionRenewedEvent(userId, eventData) {
       has_ai_add_on: hasAiAddOn,
       subscriptionId,
       payment_provider: 'recurly',
-      'customerio-integration': eventData['customerio-integration'],
     }
   )
   AnalyticsManager.setUserPropertyForUserInBackground(
@@ -291,7 +281,6 @@ async function _sendSubscriptionReactivatedEvent(userId, eventData) {
       has_ai_add_on: hasAiAddOn,
       subscriptionId,
       payment_provider: 'recurly',
-      'customerio-integration': eventData['customerio-integration'],
     }
   )
   AnalyticsManager.setUserPropertyForUserInBackground(

+ 0 - 4
services/web/frontend/js/infrastructure/event-tracking.ts

@@ -47,10 +47,6 @@ export function sendMB(key: string, segmentation: Segmentation = {}) {
     segmentation.page = window.location.pathname
   }
 
-  if (getMeta('ol-customerIoEnabled')) {
-    segmentation['customerio-integration'] = 'enabled'
-  }
-
   sendBeacon(key, segmentation)
 
   if (typeof window.gtag !== 'function') return

+ 0 - 7
services/web/test/unit/src/Subscription/RecurlyEventHandler.test.mjs

@@ -135,7 +135,6 @@ describe('RecurlyEventHandler', function () {
         has_ai_add_on: false,
         subscriptionId: ctx.eventData.subscription.uuid,
         payment_provider: 'recurly',
-        'customerio-integration': true,
       }
     )
     sinon.assert.calledWith(
@@ -191,7 +190,6 @@ describe('RecurlyEventHandler', function () {
         has_ai_add_on: false,
         subscriptionId: ctx.eventData.subscription.uuid,
         payment_provider: 'recurly',
-        'customerio-integration': true,
       }
     )
     sinon.assert.calledWith(
@@ -226,7 +224,6 @@ describe('RecurlyEventHandler', function () {
         has_ai_add_on: false,
         subscriptionId: ctx.eventData.subscription.uuid,
         payment_provider: 'recurly',
-        'customerio-integration': true,
       }
     )
     sinon.assert.calledWith(
@@ -266,7 +263,6 @@ describe('RecurlyEventHandler', function () {
         has_ai_add_on: false,
         subscriptionId: ctx.eventData.subscription.uuid,
         payment_provider: 'recurly',
-        'customerio-integration': true,
       }
     )
     sinon.assert.calledWith(
@@ -300,7 +296,6 @@ describe('RecurlyEventHandler', function () {
         has_ai_add_on: false,
         subscriptionId: ctx.eventData.subscription.uuid,
         payment_provider: 'recurly',
-        'customerio-integration': true,
       }
     )
     sinon.assert.calledWith(
@@ -339,7 +334,6 @@ describe('RecurlyEventHandler', function () {
         has_ai_add_on: false,
         subscriptionId: ctx.eventData.subscription.uuid,
         payment_provider: 'recurly',
-        'customerio-integration': true,
       }
     )
   })
@@ -359,7 +353,6 @@ describe('RecurlyEventHandler', function () {
         has_ai_add_on: false,
         subscriptionId: ctx.eventData.subscription.uuid,
         payment_provider: 'recurly',
-        'customerio-integration': true,
       }
     )
   })