Skip to content

Commit 75f970c

Browse files
committed
StatsTotalRow.swift - don't send addInsightsEvent
rename buttonTap function to match button name
1 parent 0c26781 commit 75f970c

2 files changed

Lines changed: 4 additions & 3 deletions

File tree

WordPress/Classes/ViewRelated/Stats/Shared Views/StatsTotalRow.swift

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -376,7 +376,8 @@ private extension StatsTotalRow {
376376
}
377377

378378
@IBAction func didTapDisclosureButton(_ sender: UIButton) {
379-
if let statSection = rowData?.statSection {
379+
if let statSection = rowData?.statSection,
380+
statSection != .insightsAddInsight {
380381
captureAnalyticsEventsFor(statSection)
381382
}
382383

WordPress/Classes/ViewRelated/Stats/SiteStatsDashboardViewController.swift

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,7 @@ class SiteStatsDashboardViewController: UIViewController {
7474
image: .gridicon(.cog),
7575
style: .plain,
7676
target: self,
77-
action: #selector(addStatsCardButtonTapped))
77+
action: #selector(manageInsightsButtonTapped))
7878
button.accessibilityHint = NSLocalizedString("Tap to customize insights", comment: "Accessibility hint to customize insights")
7979
return button
8080
}()
@@ -95,7 +95,7 @@ class SiteStatsDashboardViewController: UIViewController {
9595
parent?.navigationItem.rightBarButtonItem = currentSelectedPeriod == .insights ? manageInsightsButton : nil
9696
}
9797

98-
@objc func addStatsCardButtonTapped() {
98+
@objc func manageInsightsButtonTapped() {
9999
insightsTableViewController.showAddInsightView(source: "nav_bar")
100100
}
101101

0 commit comments

Comments
 (0)