Skip to content

Commit b1d76e5

Browse files
Remove remaining legacy events
1 parent d4f2d81 commit b1d76e5

File tree

4 files changed

+9
-16
lines changed

4 files changed

+9
-16
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
}

WooCommerce/src/test/kotlin/com/woocommerce/android/ui/login/jetpack/JetpackActivationRepositoryTest.kt

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
package com.woocommerce.android.ui.login.jetpack
22

3-
import com.woocommerce.android.analytics.AnalyticsTrackerWrapper
43
import com.woocommerce.android.model.JetpackConnectionStatus
54
import com.woocommerce.android.model.JetpackSiteRegistrationStatus
65
import com.woocommerce.android.tools.SelectedSite
@@ -29,15 +28,13 @@ class JetpackActivationRepositoryTest : BaseUnitTest() {
2928
}
3029
private val wooCommerceStore: WooCommerceStore = mock()
3130
private val selectedSite: SelectedSite = mock()
32-
private val analyticsTracker: AnalyticsTrackerWrapper = mock()
3331

3432
private val repository = JetpackActivationRepository(
3533
dispatcher = dispatcher,
3634
siteStore = siteStore,
3735
jetpackStore = jetpackStore,
3836
wooCommerceStore = wooCommerceStore,
39-
selectedSite = selectedSite,
40-
analyticsTrackerWrapper = analyticsTracker
37+
selectedSite = selectedSite
4138
)
4239

4340
@Test

0 commit comments

Comments
 (0)