Skip to content

Commit 28ddfe0

Browse files
Remove remaining legacy events
1 parent d4f2d81 commit 28ddfe0

File tree

3 files changed

+7
-9
lines changed

3 files changed

+7
-9
lines changed

WooCommerce/src/main/kotlin/com/woocommerce/android/analytics/AnalyticsEvent.kt

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -147,8 +147,6 @@ enum class AnalyticsEvent(override val siteless: Boolean = false) : IAnalyticsEv
147147
LOGIN_JETPACK_SITE_CREDENTIAL_RESET_PASSWORD_BUTTON_TAPPED(siteless = true),
148148
LOGIN_JETPACK_SITE_CREDENTIAL_DID_SHOW_ERROR_ALERT(siteless = true),
149149
LOGIN_JETPACK_SITE_CREDENTIAL_DID_FINISH_LOGIN(siteless = true),
150-
LOGIN_JETPACK_SETUP_CANNOT_FIND_WPCOM_USER(siteless = true),
151-
LOGIN_JETPACK_SETUP_GET_SUPPORT_BUTTON_TAPPED(siteless = true),
152150

153151
// -- Dashboard
154152
DASHBOARD_PULLED_TO_REFRESH,

WooCommerce/src/main/kotlin/com/woocommerce/android/ui/login/jetpack/JetpackActivationRepository.kt

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@ package com.woocommerce.android.ui.login.jetpack
22

33
import com.woocommerce.android.OnChangedException
44
import com.woocommerce.android.WooException
5-
import com.woocommerce.android.analytics.AnalyticsEvent
65
import com.woocommerce.android.analytics.AnalyticsTrackerWrapper
76
import com.woocommerce.android.model.JetpackConnectionStatus
87
import com.woocommerce.android.model.JetpackSiteRegistrationStatus
@@ -134,9 +133,6 @@ class JetpackActivationRepository @Inject constructor(
134133
}
135134

136135
result.data?.currentUser?.wpcomEmail.isNullOrEmpty() -> {
137-
analyticsTrackerWrapper.track(
138-
stat = AnalyticsEvent.LOGIN_JETPACK_SETUP_CANNOT_FIND_WPCOM_USER
139-
)
140136
WooLog.w(WooLog.T.LOGIN, "Cannot find Jetpack Email in response")
141137
Result.failure(JetpackMissingConnectionEmailException())
142138
}

WooCommerce/src/main/kotlin/com/woocommerce/android/ui/login/jetpack/start/JetpackActivationStartViewModel.kt

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -59,11 +59,15 @@ class JetpackActivationStartViewModel @Inject constructor(
5959

6060
fun onHelpButtonClick() {
6161
analyticsTrackerWrapper.track(
62-
stat = AnalyticsEvent.LOGIN_JETPACK_SETUP_GET_SUPPORT_BUTTON_TAPPED,
62+
stat = AnalyticsEvent.JETPACK_SETUP_FLOW,
6363
properties = mapOf(
64-
AnalyticsTracker.KEY_JETPACK_INSTALLATION_STEP to
64+
AnalyticsTracker.KEY_STEP to if (navArgs.jetpackStatus.isJetpackInstalled) {
6565
JetpackActivationMainViewModel.StepType.Connection.analyticsName
66-
),
66+
} else {
67+
JetpackActivationMainViewModel.StepType.Installation.analyticsName
68+
},
69+
AnalyticsTracker.KEY_TAP to AnalyticsTracker.VALUE_JETPACK_SETUP_TAP_SUPPORT,
70+
)
6771
)
6872
triggerEvent(NavigateToHelpScreen(JETPACK_INSTALLATION))
6973
}

0 commit comments

Comments
 (0)