Skip to content

Commit 1c2e6b0

Browse files
authored
Merge pull request #9820 from woocommerce/issue/track-learn-more-tapped-everywher
[Mobile Payments] Track when Learn More is tapped on Set up Tap to Pay and Manage Card Reader screens
2 parents 6724eb6 + 0788623 commit 1c2e6b0

File tree

3 files changed

+28
-19
lines changed

3 files changed

+28
-19
lines changed

WooCommerce/Classes/Analytics/WooAnalyticsEvent.swift

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1893,13 +1893,19 @@ extension WooAnalyticsEvent {
18931893
enum LearnMoreLinkSource {
18941894
case paymentsMenu
18951895
case paymentMethods
1896+
case tapToPaySummary
1897+
case manageCardReader
18961898

18971899
var trackingValue: String {
18981900
switch self {
18991901
case .paymentsMenu:
19001902
return "payments_menu"
19011903
case .paymentMethods:
19021904
return "payment_methods"
1905+
case .tapToPaySummary:
1906+
return "tap_to_pay_summary"
1907+
case .manageCardReader:
1908+
return "manage_card_reader"
19031909
}
19041910
}
19051911
}

WooCommerce/Classes/ViewRelated/Dashboard/Settings/CardReadersV2/CardReaderSettingsSearchingViewController.swift

Lines changed: 19 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -147,28 +147,29 @@ struct CardReaderSettingsSearchingView: View {
147147
.buttonStyle(PrimaryButtonStyle())
148148
.padding(.bottom, 8)
149149

150-
InPersonPaymentsLearnMore()
151-
.customOpenURL(action: { url in
152-
switch url {
153-
case LearnMoreViewModel.learnMoreURL:
154-
if let url = learnMoreUrl {
155-
showURL?(url)
156-
}
157-
default:
150+
InPersonPaymentsLearnMore(viewModel: LearnMoreViewModel(
151+
tappedAnalyticEvent: WooAnalyticsEvent.InPersonPayments.learnMoreTapped(source: .manageCardReader)))
152+
.customOpenURL(action: { url in
153+
switch url {
154+
case LearnMoreViewModel.learnMoreURL:
155+
if let url = learnMoreUrl {
158156
showURL?(url)
159157
}
160-
})
161-
}
162-
.frame(
163-
maxWidth: .infinity,
164-
maxHeight: .infinity
165-
)
166-
.padding()
167-
.if(isCompact || isSizeCategoryLargeThanExtraLarge) {content in
168-
ScrollView(.vertical) {
169-
content
158+
default:
159+
showURL?(url)
170160
}
161+
})
162+
}
163+
.frame(
164+
maxWidth: .infinity,
165+
maxHeight: .infinity
166+
)
167+
.padding()
168+
.if(isCompact || isSizeCategoryLargeThanExtraLarge) {content in
169+
ScrollView(.vertical) {
170+
content
171171
}
172+
}
172173
}
173174
}
174175

WooCommerce/Classes/ViewRelated/Dashboard/Settings/CardReadersV2/SetUpTapToPayInformationViewController.swift

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -120,7 +120,9 @@ struct SetUpTapToPayInformationView: View {
120120
.disabled(!viewModel.enableSetup)
121121

122122
InPersonPaymentsLearnMore(
123-
viewModel: LearnMoreViewModel(formatText: Localization.learnMore))
123+
viewModel: LearnMoreViewModel(
124+
formatText: Localization.learnMore,
125+
tappedAnalyticEvent: WooAnalyticsEvent.InPersonPayments.learnMoreTapped(source: .tapToPaySummary)))
124126
.customOpenURL(action: { url in
125127
switch url {
126128
case LearnMoreViewModel.learnMoreURL:

0 commit comments

Comments
 (0)