Track Reason for Cash on Delivery Learn More event

Author: joshheald

WooCommerce/Classes/Analytics/WooAnalyticsEvent.swift

@@ -1137,9 +1137,12 @@ extension WooAnalyticsEvent {
         ///
         /// - Parameter countryCode: the country code of the store.
         ///
-        static func cardPresentOnboardingLearnMoreTapped(countryCode: String) -> WooAnalyticsEvent {
+        static func cardPresentOnboardingLearnMoreTapped(reason: String, countryCode: String) -> WooAnalyticsEvent {
             WooAnalyticsEvent(statName: .cardPresentOnboardingLearnMoreTapped,
-                              properties: [Keys.countryCode: countryCode])
+                              properties: [
+                                Keys.countryCode: countryCode,
+                                Keys.reason: reason
+                              ])
         }
 
         /// Tracked when the In-Person Payments onboarding cannot be completed for some reason.

WooCommerce/Classes/ViewRelated/Dashboard/Settings/In-Person Payments/InPersonPaymentsLearnMore.swift

@@ -7,13 +7,16 @@ struct InPersonPaymentsLearnMore: View {
     let url: URL
     let linkText: String
     let formatText: String
+    let analyticReason: String?
 
     init(url: URL = learnMoreURL,
          linkText: String = Localization.learnMoreLink,
-         formatText: String = Localization.learnMoreText) {
+         formatText: String = Localization.learnMoreText,
+         analyticReason: String? = nil) {
         self.url = url
         self.linkText = linkText
         self.formatText = formatText
+        self.analyticReason = analyticReason
     }
 
     private let cardPresentConfiguration = CardPresentConfigurationLoader().configuration
@@ -28,8 +31,10 @@ struct InPersonPaymentsLearnMore: View {
         }
             .padding(.vertical, Constants.verticalPadding)
             .onTapGesture {
-                ServiceLocator.analytics.track(event: WooAnalyticsEvent.InPersonPayments
-                                                .cardPresentOnboardingLearnMoreTapped(countryCode: cardPresentConfiguration.countryCode))
+                ServiceLocator.analytics.track(
+                    event: WooAnalyticsEvent.InPersonPayments.cardPresentOnboardingLearnMoreTapped(
+                        reason: analyticReason ?? "",
+                        countryCode: cardPresentConfiguration.countryCode))
                 customOpenURL?(url)
             }
     }

WooCommerce/Classes/ViewRelated/Dashboard/Settings/In-Person Payments/Onboarding Errors/InPersonPaymentsCashOnDeliveryPaymentGatewayNotSetUpView.swift

@@ -29,7 +29,8 @@ struct InPersonPaymentsCashOnDeliveryPaymentGatewayNotSetUpView: View {
             Spacer()
 
             InPersonPaymentsLearnMore(url: WooConstants.URLs.cashOnDeliveryLearnMoreUrl.asURL(),
-                                      formatText: Localization.cashOnDeliveryLearnMore)
+                                      formatText: Localization.cashOnDeliveryLearnMore,
+                                      analyticReason: viewModel.analyticReason)
         }
     }
 }

WooCommerce/Classes/ViewRelated/Dashboard/Settings/In-Person Payments/Onboarding Errors/InPersonPaymentsCashOnDeliveryPaymentGatewayNotSetUpViewModel.swift

@@ -36,6 +36,8 @@ final class InPersonPaymentsCashOnDeliveryPaymentGatewayNotSetUpViewModel: Obser
 
     @Published var awaitingResponse = false
 
+    let analyticReason: String = CardPresentPaymentOnboardingState.codPaymentGatewayNotSetUp.reasonForAnalytics ?? ""
+
     // MARK: - Configuration properties
     private let cardPresentPaymentsConfiguration: CardPresentPaymentsConfiguration
 
@@ -114,19 +116,15 @@ final class InPersonPaymentsCashOnDeliveryPaymentGatewayNotSetUpViewModel: Obser
 private extension InPersonPaymentsCashOnDeliveryPaymentGatewayNotSetUpViewModel {
     typealias Event = WooAnalyticsEvent.InPersonPayments
 
-    var reason: String {
-        CardPresentPaymentOnboardingState.codPaymentGatewayNotSetUp.reasonForAnalytics ?? ""
-    }
-
     func trackSkipTapped() {
-        let event = Event.cardPresentOnboardingStepSkipped(reason: reason,
+        let event = Event.cardPresentOnboardingStepSkipped(reason: analyticReason,
                                                            remindLater: false,
                                                            countryCode: cardPresentPaymentsConfiguration.countryCode)
         analytics.track(event: event)
     }
 
     func trackEnableTapped() {
-        let event = Event.cardPresentOnboardingCtaTapped(reason: reason,
+        let event = Event.cardPresentOnboardingCtaTapped(reason: analyticReason,
                                                          countryCode: cardPresentPaymentsConfiguration.countryCode)
         analytics.track(event: event)
     }