Skip to content

Commit 14d3f01

Browse files
committed
Update tracking for connection type
1 parent 98b1a13 commit 14d3f01

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

WooCommerce/Classes/Authentication/Jetpack Setup/Native Jetpack Setup/JetpackSetupViewModel.swift

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@ final class JetpackSetupViewModel: ObservableObject {
1717
private let storeNavigationHandler: (_ connectedEmail: String?) -> Void
1818
private let wpcomCredentials: Credentials?
1919
private var isPluginActivated = false
20+
private var connectionType = WooAnalyticsEvent.JetpackSetup.ConnectionType.native
2021

2122
@Published private(set) var setupSteps: [JetpackInstallStep]
2223

@@ -175,7 +176,6 @@ final class JetpackSetupViewModel: ObservableObject {
175176

176177
/// Tracks events if the current flow is Jetpack setup after login with site credentials
177178
func trackSetup(tap: WooAnalyticsEvent.JetpackSetup.SetupFlow.TapTarget? = nil,
178-
connectionType: WooAnalyticsEvent.JetpackSetup.ConnectionType = .native,
179179
failure: Error? = nil) {
180180
/// Helper for analytics since `currentSetupStep` is optional.
181181
let currentStepForAnalytics: JetpackInstallStep = currentSetupStep ?? (connectionOnly ? .connection : .installation)
@@ -262,7 +262,8 @@ private extension JetpackSetupViewModel {
262262
///
263263
func startConnectionWithWebView() {
264264
currentSetupStep = .connection
265-
trackSetup(connectionType: .web)
265+
connectionType = .web
266+
trackSetup()
266267
let action = JetpackConnectionAction.fetchJetpackConnectionURL { [weak self] result in
267268
guard let self else { return }
268269
switch result {
@@ -277,7 +278,7 @@ private extension JetpackSetupViewModel {
277278
}
278279
self.shouldPresentWebView = true
279280
case .failure(let error):
280-
self.trackSetup(connectionType: .web, failure: error)
281+
self.trackSetup(failure: error)
281282
DDLogError("⛔️ Error fetching Jetpack connection URL: \(error)")
282283
self.setupError = error
283284
self.setupFailed = true

0 commit comments

Comments
 (0)