Merge pull request #19786 from wordpress-mobile/task/19450-phase-3-card-analytics

Jetpack Focus: Add analytics tracking for the Jetpack menu card

Author: hassaanelgarem

WordPress/Classes/Utility/Analytics/WPAnalyticsEvent.swift

@@ -414,6 +414,12 @@ import Foundation
     case jetpackSiteCreationOverlayDisplayed
     case jetpackSiteCreationOverlayButtonTapped
     case jetpackSiteCreationOverlayDismissed
+    case jetpackBrandingMenuCardDisplayed
+    case jetpackBrandingMenuCardTapped
+    case jetpackBrandingMenuCardLinkTapped
+    case jetpackBrandingMenuCardHidden
+    case jetpackBrandingMenuCardRemindLater
+    case jetpackBrandingMenuCardContextualMenuAccessed
 
     /// A String that represents the event
     var value: String {
@@ -1123,6 +1129,18 @@ import Foundation
             return "remove_site_creation_overlay_button_tapped"
         case .jetpackSiteCreationOverlayDismissed:
             return "remove_site_creation_overlay_dismissed"
+        case .jetpackBrandingMenuCardDisplayed:
+            return "remove_feature_card_displayed"
+        case .jetpackBrandingMenuCardTapped:
+            return "remove_feature_card_tapped"
+        case .jetpackBrandingMenuCardLinkTapped:
+            return "remove_feature_card_link_tapped"
+        case .jetpackBrandingMenuCardHidden:
+            return "remove_feature_card_hide_tapped"
+        case .jetpackBrandingMenuCardRemindLater:
+            return "remove_feature_card_remind_later_tapped"
+        case .jetpackBrandingMenuCardContextualMenuAccessed:
+            return "remove_feature_card_menu_accessed"
 
         } // END OF SWITCH
     }

WordPress/Classes/ViewRelated/Jetpack/Branding/Menu Card/BlogDetailsViewController+JetpackBrandingMenuCard.swift

@@ -10,7 +10,9 @@ extension BlogDetailsViewController {
     @objc func jetpackCardSectionViewModel() -> BlogDetailsSection {
         let row = BlogDetailsRow()
         row.callback = {
+            let presenter = JetpackBrandingMenuCardPresenter()
             JetpackFeaturesRemovalCoordinator.presentOverlayIfNeeded(from: .card, in: self)
+            presenter.trackCardTapped()
         }
 
         let section = BlogDetailsSection(title: nil,

WordPress/Classes/ViewRelated/Jetpack/Branding/Menu Card/JetpackBrandingMenuCardCell.swift

@@ -23,8 +23,8 @@ class JetpackBrandingMenuCardCell: UITableViewCell {
         frameView.configureButtonContainerStackView()
         frameView.hideHeader()
 
-        frameView.onEllipsisButtonTap = {
-            // TODO: Track menu shown
+        frameView.onEllipsisButtonTap = { [weak self] in
+            self?.presenter.trackContexualMenuAccessed()
         }
         frameView.ellipsisButton.showsMenuAsPrimaryAction = true
         frameView.ellipsisButton.menu = contextMenu
@@ -134,7 +134,8 @@ class JetpackBrandingMenuCardCell: UITableViewCell {
     private func commonInit() {
         setupViews()
         setupContent()
-        // TODO: Track card shown
+
+        presenter.trackCardShown()
     }
 
     // MARK: Helpers
@@ -164,7 +165,7 @@ class JetpackBrandingMenuCardCell: UITableViewCell {
         let webViewController = WebViewControllerFactory.controller(url: url, source: Constants.analyticsSource)
         let navController = UINavigationController(rootViewController: webViewController)
         viewController?.present(navController, animated: true)
-        // TODO: Track button tapped
+        presenter.trackLinkTapped()
     }
 }
 
@@ -191,13 +192,11 @@ private extension JetpackBrandingMenuCardCell {
     private func remindMeLaterTapped() {
         presenter.remindLaterTapped()
         viewController?.reloadTableView()
-        // TODO: Track button tapped
     }
 
     private func hideThisTapped() {
         presenter.hideThisTapped()
         viewController?.reloadTableView()
-        // TODO: Track button tapped
     }
 }
 

WordPress/Classes/ViewRelated/Jetpack/Branding/Menu Card/JetpackBrandingMenuCardPresenter.swift

@@ -11,8 +11,8 @@ class JetpackBrandingMenuCardPresenter {
 
     private let remoteConfigStore: RemoteConfigStore
     private let persistenceStore: UserPersistentRepository
-    private let featureFlagStore: RemoteFeatureFlagStore
     private let currentDateProvider: CurrentDateProvider
+    private let featureFlagStore: RemoteFeatureFlagStore
 
     // MARK: Initializers
 
@@ -21,9 +21,9 @@ class JetpackBrandingMenuCardPresenter {
          persistenceStore: UserPersistentRepository = UserDefaults.standard,
          currentDateProvider: CurrentDateProvider = DefaultCurrentDateProvider()) {
         self.remoteConfigStore = remoteConfigStore
-        self.featureFlagStore = featureFlagStore
         self.persistenceStore = persistenceStore
         self.currentDateProvider = currentDateProvider
+        self.featureFlagStore = featureFlagStore
     }
 
     // MARK: Public Functions
@@ -55,10 +55,46 @@ class JetpackBrandingMenuCardPresenter {
         let duration = Constants.remindLaterDurationInDays * Constants.secondsInDay
         let newDate = now.addingTimeInterval(TimeInterval(duration))
         showCardOnDate = newDate
+        trackRemindMeLaterTapped()
     }
 
     func hideThisTapped() {
         shouldHideCard = true
+        trackHideThisTapped()
+    }
+}
+
+// MARK: Analytics
+
+extension JetpackBrandingMenuCardPresenter {
+
+    func trackCardShown() {
+        WPAnalytics.track(.jetpackBrandingMenuCardDisplayed, properties: analyticsProperties)
+    }
+
+    func trackLinkTapped() {
+        WPAnalytics.track(.jetpackBrandingMenuCardLinkTapped, properties: analyticsProperties)
+    }
+
+    func trackCardTapped() {
+        WPAnalytics.track(.jetpackBrandingMenuCardTapped, properties: analyticsProperties)
+    }
+
+    func trackContexualMenuAccessed() {
+        WPAnalytics.track(.jetpackBrandingMenuCardContextualMenuAccessed, properties: analyticsProperties)
+    }
+
+    func trackHideThisTapped() {
+        WPAnalytics.track(.jetpackBrandingMenuCardHidden, properties: analyticsProperties)
+    }
+
+    func trackRemindMeLaterTapped() {
+        WPAnalytics.track(.jetpackBrandingMenuCardRemindLater, properties: analyticsProperties)
+    }
+
+    private var analyticsProperties: [String: String] {
+        let phase = JetpackFeaturesRemovalCoordinator.generalPhase(featureFlagStore: featureFlagStore)
+        return [Constants.phaseAnalyticsKey: phase.rawValue]
     }
 }
 
@@ -90,6 +126,7 @@ private extension JetpackBrandingMenuCardPresenter {
         static let remindLaterDurationInDays = 4
         static let shouldHideCardKey = "JetpackBrandingShouldHideCardKey"
         static let showCardOnDateKey = "JetpackBrandingShowCardOnDateKey"
+        static let phaseAnalyticsKey = "phase"
     }
 
     enum Strings {