Skip to content

Commit 98ab309

Browse files
Remove remaining legacy events
1 parent d4f2d81 commit 98ab309

File tree

3 files changed

+8
-12
lines changed

3 files changed

+8
-12
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: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +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
6-
import com.woocommerce.android.analytics.AnalyticsTrackerWrapper
75
import com.woocommerce.android.model.JetpackConnectionStatus
86
import com.woocommerce.android.model.JetpackSiteRegistrationStatus
97
import com.woocommerce.android.tools.SelectedSite
@@ -26,8 +24,7 @@ class JetpackActivationRepository @Inject constructor(
2624
private val siteStore: SiteStore,
2725
private val jetpackStore: JetpackStore,
2826
private val wooCommerceStore: WooCommerceStore,
29-
private val selectedSite: SelectedSite,
30-
private val analyticsTrackerWrapper: AnalyticsTrackerWrapper
27+
private val selectedSite: SelectedSite
3128
) {
3229
companion object {
3330
private const val DEFAULT_MAX_RETRY = 2
@@ -134,9 +131,6 @@ class JetpackActivationRepository @Inject constructor(
134131
}
135132

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

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)