Merge pull request #6189 from woocommerce/issue/6188-tracks-m1

Coupons: Add tracks for Milestone 1

Author: itsmeichigo

WooCommerce/Classes/Analytics/WooAnalyticsStat.swift

@@ -565,6 +565,12 @@ public enum WooAnalyticsStat: String {
     case hubMenuSwitchStoreTapped = "hub_menu_switch_store_tapped"
     case hubMenuOptionTapped = "hub_menu_option_tapped"
     case hubMenuSettingsTapped = "hub_menu_settings_tapped"
+
+    // MARK: Coupons
+    case couponsLoaded = "coupons_loaded"
+    case couponsLoadedFailed = "coupons_loaded_failed"
+    case couponsListSearchTapped = "coupons_list_search_tapped"
+    case couponDetails = "coupon_details"
 }
 
 public extension WooAnalyticsStat {

WooCommerce/Classes/ViewRelated/Coupons/CouponDetails/CouponDetails.swift

@@ -34,6 +34,8 @@ struct CouponDetails: View {
         self.noticePresenter = DefaultNoticePresenter()
         viewModel.syncCoupon()
         viewModel.loadCouponReport()
+
+        ServiceLocator.analytics.track(.couponDetails, withProperties: ["action": "loaded"])
     }
 
     private var detailRows: [DetailRow] {
@@ -60,9 +62,11 @@ struct CouponDetails: View {
                                         UIPasteboard.general.string = viewModel.couponCode
                                         let notice = Notice(title: Localization.couponCopied, feedbackType: .success)
                                         noticePresenter.enqueue(notice: notice)
+                                        ServiceLocator.analytics.track(.couponDetails, withProperties: ["action": "copied_code"])
                                     }),
                                     .default(Text(Localization.shareCoupon), action: {
                                         showingShareSheet = true
+                                        ServiceLocator.analytics.track(.couponDetails, withProperties: ["action": "shared_code"])
                                     }),
                                     .cancel()
                                 ]

WooCommerce/Classes/ViewRelated/Coupons/CouponListViewController.swift

@@ -167,7 +167,7 @@ private extension CouponListViewController {
     /// Shows `SearchViewController`.
     ///
     @objc private func displaySearchCoupons() {
-        // TODO: add analytics
+        ServiceLocator.analytics.track(.couponsListSearchTapped)
         let searchViewController = SearchViewController<TitleAndSubtitleAndStatusTableViewCell, CouponSearchUICommand>(
             storeID: siteID,
             command: CouponSearchUICommand(),

WooCommerce/Classes/ViewRelated/Coupons/CouponListViewModel.swift

@@ -151,20 +151,23 @@ extension CouponListViewModel: SyncingCoordinatorDelegate {
                                 pageNumber: pageNumber,
                                 pageSize: pageSize) { [weak self] result in
                 guard let self = self else { return }
-                self.handleCouponSyncResult(result: result)
+                self.handleCouponSyncResult(result: result, pageNumber: pageNumber)
                 onCompletion?(result.isSuccess)
         }
 
         storesManager.dispatch(action)
     }
 
-    func handleCouponSyncResult(result: Result<Bool, Error>) {
+    func handleCouponSyncResult(result: Result<Bool, Error>, pageNumber: Int) {
         switch result {
         case .success:
             DDLogInfo("Synchronized coupons")
+            ServiceLocator.analytics.track(.couponsLoaded,
+                                           withProperties: ["is_loading_more": pageNumber != SyncingCoordinator.Defaults.pageFirstIndex])
 
         case .failure(let error):
             DDLogError("⛔️ Error synchronizing coupons: \(error)")
+            ServiceLocator.analytics.track(.couponsLoadedFailed, withError: error)
         }
 
         self.transitionToResultsUpdatedState(hasData: couponViewModels.isNotEmpty)

WooCommerce/WooCommerceTests/ViewRelated/Coupons/CouponListViewModelTests.swift

@@ -97,7 +97,7 @@ final class CouponListViewModelTests: XCTestCase {
         setUpWithCouponFetched()
 
         // When
-        sut.handleCouponSyncResult(result: .success(false))
+        sut.handleCouponSyncResult(result: .success(false), pageNumber: 1)
 
         // Then
         XCTAssertEqual(sut.state, .coupons)
@@ -106,7 +106,7 @@ final class CouponListViewModelTests: XCTestCase {
 
     func test_handleCouponSyncResult_shows_empty_when_no_coupons_present() {
         // When
-        sut.handleCouponSyncResult(result: .success(false))
+        sut.handleCouponSyncResult(result: .success(false), pageNumber: 1)
 
         // Then
         XCTAssertEqual(sut.state, .empty)
@@ -140,7 +140,7 @@ final class CouponListViewModelTests: XCTestCase {
         sut.refreshCoupons()
 
         // When
-        sut.handleCouponSyncResult(result: .success(false))
+        sut.handleCouponSyncResult(result: .success(false), pageNumber: 1)
 
         // Then
         XCTAssertEqual(sut.state, .coupons)