[Woo POS] Coupons: Enable site coupons when tap on CTA

WooCommerce/Classes/POS/Controllers/PointOfSaleCouponsController.swift

@@ -5,7 +5,7 @@ import protocol Yosemite.PointOfSaleItemServiceProtocol
 import protocol Yosemite.PointOfSaleCouponServiceProtocol
 
 @available(iOS 17.0, *)
-@Observable final class PointOfSaleCouponsController: PointOfSaleItemsControllerProtocol {
+@Observable final class PointOfSaleCouponsController: PointOfSaleCouponsControllerProtocol {
     var itemsViewState: ItemsViewState = ItemsViewState(containerState: .loading,
                                                         itemsStack: ItemsStackState(root: .loading([]),
                                                                                     itemStates: [:]))
@@ -39,6 +39,19 @@ import protocol Yosemite.PointOfSaleCouponServiceProtocol
         // Pagination https://github.com/woocommerce/woocommerce-ios/issues/15343
         await loadFirstPage()
     }
+
+    @MainActor
+    func enableCoupons() async {
+        // TODO: WOOMOB-255
+        // Handle loading state while coupons are being enabled
+        do {
+            try await couponProvider.enableCoupons()
+        } catch {
+            // TODO: WOOMOB-267
+            // Handle error when failed to enable, and allow retry action
+            debugPrint(error)
+        }
+    }
 }
 
 @available(iOS 17.0, *)

WooCommerce/Classes/POS/Controllers/PointOfSaleItemsController.swift

@@ -26,6 +26,12 @@ protocol PointOfSaleItemsControllerProtocol {
     func loadNextItems(base: ItemListBaseItem) async
 }
 
+@available(iOS 17.0, *)
+protocol PointOfSaleCouponsControllerProtocol: PointOfSaleItemsControllerProtocol {
+    /// Enables coupons in store settings, if needed
+    func enableCoupons() async
+}
+
 @available(iOS 17.0, *)
 protocol PointOfSaleSearchingItemsControllerProtocol: PointOfSaleItemsControllerProtocol {
     /// Searches for items

WooCommerce/Classes/POS/Models/PointOfSaleAggregateModel.swift

@@ -25,6 +25,7 @@ protocol PointOfSaleAggregateModelProtocol {
     func trackCardPaymentsOnboardingShown()
 
     var itemsViewState: ItemsViewState { get }
+
     func loadItems(base: ItemListBaseItem) async
     func loadNextItems(base: ItemListBaseItem) async
 
@@ -56,14 +57,15 @@ protocol PointOfSaleAggregateModelProtocol {
 
     var itemsViewState: ItemsViewState { itemsController.itemsViewState }
     var couponsViewState: ItemsViewState { couponsController.itemsViewState }
+    var currentViewState: ItemsViewState
 
     private(set) var cart: Cart = .init()
 
     var orderState: PointOfSaleOrderState { orderController.orderState.externalState }
     private var internalOrderState: PointOfSaleInternalOrderState { orderController.orderState }
 
     private let itemsController: PointOfSaleItemsControllerProtocol
-    private let couponsController: PointOfSaleItemsControllerProtocol
+    private let couponsController: PointOfSaleCouponsControllerProtocol
 
     private let cardPresentPaymentService: CardPresentPaymentFacade
     private let orderController: PointOfSaleOrderControllerProtocol
@@ -76,7 +78,7 @@ protocol PointOfSaleAggregateModelProtocol {
     private var cancellables: Set<AnyCancellable> = []
 
     init(itemsController: PointOfSaleItemsControllerProtocol,
-         couponsController: PointOfSaleItemsControllerProtocol,
+         couponsController: PointOfSaleCouponsControllerProtocol,
          cardPresentPaymentService: CardPresentPaymentFacade,
          orderController: PointOfSaleOrderControllerProtocol,
          analytics: Analytics = ServiceLocator.analytics,
@@ -89,6 +91,9 @@ protocol PointOfSaleAggregateModelProtocol {
         self.analytics = analytics
         self.collectOrderPaymentAnalyticsTracker = collectOrderPaymentAnalyticsTracker
         self.paymentState = paymentState
+        // Initial, set to items (products)
+        self.currentViewState = itemsController.itemsViewState
+
         publishCardReaderConnectionStatus()
         publishPaymentMessages()
         setupReaderReconnectionObservation()
@@ -98,22 +103,33 @@ protocol PointOfSaleAggregateModelProtocol {
 // MARK: - ItemList
 @available(iOS 17.0, *)
 extension PointOfSaleAggregateModel {
+    func updateCurrentViewState(base: ItemListBaseItem) {
+        let viewState = base.itemType == .products ? itemsViewState : couponsViewState
+        currentViewState = viewState
+    }
+
     @MainActor
     func loadItems(base: ItemListBaseItem) async {
         let controller = base.itemType == .products ? itemsController : couponsController
+
         await controller.loadItems(base: base)
+        updateCurrentViewState(base: base)
     }
 
     @MainActor
     func refreshItems(base: ItemListBaseItem) async {
         let controller = base.itemType == .products ? itemsController : couponsController
+
         await controller.refreshItems(base: base)
+        updateCurrentViewState(base: base)
     }
 
     @MainActor
     func loadNextItems(base: ItemListBaseItem) async {
         let controller = base.itemType == .products ? itemsController : couponsController
+
         await controller.loadNextItems(base: base)
+        updateCurrentViewState(base: base)
     }
 }
 
@@ -157,6 +173,14 @@ extension PointOfSaleAggregateModel {
     }
 }
 
+// MARK: - Coupons
+@available(iOS 17.0, *)
+extension PointOfSaleAggregateModel {
+    func enableCoupons() async {
+        await couponsController.enableCoupons()
+    }
+}
+
 // MARK: - Track events
 @available(iOS 17.0, *)
 private extension PointOfSaleAggregateModel {

WooCommerce/Classes/POS/Models/PointOfSaleErrorState.swift

@@ -1,49 +1,108 @@
 import Foundation
 
 struct PointOfSaleErrorState: Equatable {
+    enum ErrorType: Equatable {
+        case productsLoadError
+        case variationsLoadError
+        case productsNextPageError
+        case variationsNextPageError
+        case couponsNotFound
+        case couponsLoadError
+        case couponsDisabled
+    }
+
+    let errorType: ErrorType
     let title: String
     let subtitle: String
     let buttonText: String
 
     static func errorOnLoadingProducts() -> Self {
-        PointOfSaleErrorState(title: Constants.failedToLoadProductsTitle,
-                              subtitle: Constants.failedToLoadProductsSubtitle,
-                              buttonText: Constants.failedToLoadProductsButtonTitle)
+        PointOfSaleErrorState(
+            errorType: .productsLoadError,
+            title: Constants.failedToLoadProductsTitle,
+            subtitle: Constants.failedToLoadProductsSubtitle,
+            buttonText: Constants.failedToLoadProductsButtonTitle)
     }
 
     static func errorOnLoadingVariations() -> Self {
-        PointOfSaleErrorState(title: Constants.failedToLoadVariationsTitle,
-                              subtitle: Constants.failedToLoadVariationsSubtitle,
-                              buttonText: Constants.failedToLoadVariationsButtonTitle)
+        PointOfSaleErrorState(
+            errorType: .variationsLoadError,
+            title: Constants.failedToLoadVariationsTitle,
+            subtitle: Constants.failedToLoadVariationsSubtitle,
+            buttonText: Constants.failedToLoadVariationsButtonTitle)
     }
 
     static func errorOnLoadingProductsNextPage() -> Self {
-        PointOfSaleErrorState(title: Constants.failedToLoadProductsNextPageTitle,
-                              subtitle: Constants.failedToLoadProductsNextPageSubtitle,
-                              buttonText: Constants.failedToLoadProductsNextPageButtonTitle)
+        PointOfSaleErrorState(
+            errorType: .productsNextPageError,
+            title: Constants.failedToLoadProductsNextPageTitle,
+            subtitle: Constants.failedToLoadProductsNextPageSubtitle,
+            buttonText: Constants.failedToLoadProductsNextPageButtonTitle)
     }
 
     static func errorOnLoadingVariationsNextPage() -> Self {
-        PointOfSaleErrorState(title: Constants.failedToLoadVariationsNextPageTitle,
-                              subtitle: Constants.failedToLoadVariationsNextPageSubtitle,
-                              buttonText: Constants.failedToLoadVariationsNextPageButtonTitle)
+        PointOfSaleErrorState(
+            errorType: .variationsNextPageError,
+            title: Constants.failedToLoadVariationsNextPageTitle,
+            subtitle: Constants.failedToLoadVariationsNextPageSubtitle,
+            buttonText: Constants.failedToLoadVariationsNextPageButtonTitle)
     }
 
     static func errorCouponsNotFound() -> Self {
-        PointOfSaleErrorState(title: Constants.noCouponsFoundTitle,
-                              subtitle: Constants.noCouponsFoundSubtitle,
-                              buttonText: Constants.noCouponsFoundButtonTitle)
+        PointOfSaleErrorState(
+            errorType: .couponsNotFound,
+            title: Constants.noCouponsFoundTitle,
+            subtitle: Constants.noCouponsFoundSubtitle,
+            buttonText: Constants.noCouponsFoundButtonTitle)
     }
 
     static func errorOnLoadingCoupons() -> Self {
-        PointOfSaleErrorState(title: "Error loading coupons", subtitle: "Error loading coupons", buttonText: "Retry")
+        PointOfSaleErrorState(
+            errorType: .couponsLoadError,
+            title: Constants.loadingCouponsErrorTitle,
+            subtitle: Constants.loadingCouponsErrorSubtitle,
+            buttonText: Constants.loadingCouponsErrorRetry)
     }
 
     static func errorCouponsDisabled() -> Self {
-        PointOfSaleErrorState(title: "Error loading coupons", subtitle: "Please enable coupons in WooCommerce Settings, and tap Retry", buttonText: "Retry")
+        PointOfSaleErrorState(
+            errorType: .couponsDisabled,
+            title: Constants.loadingCouponsDisabledTitle,
+            subtitle: Constants.loadingCouponsDisabledSubtitle,
+            buttonText: Constants.loadingCouponsDisabledAction)
     }
 
     enum Constants {
+        static let loadingCouponsErrorTitle = NSLocalizedString(
+            "pos.itemList.loadingCouponsErrorTitle",
+            value: "Error loading coupons",
+            comment: "Title appearing on the coupon list screen when there's an error loading coupons."
+        )
+        static let loadingCouponsErrorSubtitle = NSLocalizedString(
+            "pos.itemList.loadingCouponsErrorSubtitle",
+            value: "Error loading coupons",
+            comment: "Subtitle appearing on the coupon list screen when there's an error loading coupons."
+        )
+        static let loadingCouponsErrorRetry = NSLocalizedString(
+            "pos.itemList.loadingCouponsErrorRetry",
+            value: "Retry",
+            comment: "Text of the button appearing on the coupon list screen when there's an error loading coupons."
+        )
+        static let loadingCouponsDisabledTitle = NSLocalizedString(
+            "pos.itemList.loadingCouponsDisabledTitle",
+            value: "Error loading coupons",
+            comment: "Title appearing on the coupon list screen when coupons are disabled."
+        )
+        static let loadingCouponsDisabledSubtitle = NSLocalizedString(
+            "pos.itemList.loadingCouponsDisabledSubtitle",
+            value: "Please enable the use of coupon codes in your store.",
+            comment: "Subtitle appearing on the coupon list screen when coupons are disabled."
+        )
+        static let loadingCouponsDisabledAction = NSLocalizedString(
+            "pos.itemList.loadingCouponsDisabledAction",
+            value: "Enable",
+            comment: "Text of the button appearing on the coupon list screen when coupons are disabled."
+        )
         static let noCouponsFoundTitle = NSLocalizedString(
             "pos.itemList.noCouponsFoundTitle",
             value: "No coupons found",

WooCommerce/Classes/POS/Presentation/CardReaderConnection/CardReaderConnectionStatusView.swift

@@ -171,7 +171,7 @@ private extension CardReaderConnectionStatusView {
 #Preview {
     let posModel = PointOfSaleAggregateModel(
         itemsController: PointOfSalePreviewItemsController(),
-        couponsController: PointOfSalePreviewItemsController(),
+        couponsController: PointOfSalePreviewCouponsController(),
         cardPresentPaymentService: CardPresentPaymentPreviewService(),
         orderController: PointOfSalePreviewOrderController(),
         collectOrderPaymentAnalyticsTracker: POSCollectOrderPaymentAnalytics()

WooCommerce/Classes/POS/Presentation/CardReaderConnection/UI States/PointOfSaleItemListErrorView.swift

@@ -3,11 +3,11 @@ import SwiftUI
 /// A view that displays an error message with a retry CTA when the list of POS items fails to load.
 struct PointOfSaleItemListErrorView: View {
     private let error: PointOfSaleErrorState
-    private let onRetry: (() -> Void)?
+    private let onAction: (() -> Void)?
 
-    init(error: PointOfSaleErrorState, onRetry: (() -> Void)? = nil) {
+    init(error: PointOfSaleErrorState, onAction: (() -> Void)? = nil) {
         self.error = error
-        self.onRetry = onRetry
+        self.onAction = onAction
     }
 
     var body: some View {
@@ -33,7 +33,7 @@ struct PointOfSaleItemListErrorView: View {
                 Spacer().frame(height: PointOfSaleCardPresentPaymentLayout.textAndButtonSpacing)
 
                 Button(action: {
-                    onRetry?()
+                    onAction?()
                 }, label: {
                     Text(error.buttonText)
                 })
@@ -47,5 +47,5 @@ struct PointOfSaleItemListErrorView: View {
 }
 
 #Preview {
-    PointOfSaleItemListErrorView(error: .errorOnLoadingProducts(), onRetry: nil)
+    PointOfSaleItemListErrorView(error: .errorOnLoadingProducts(), onAction: nil)
 }

WooCommerce/Classes/POS/Presentation/CardReaderConnection/UI States/PointOfSaleItemListFullscreenErrorView.swift

@@ -3,20 +3,20 @@ import SwiftUI
 /// A view that displays an error message with a retry CTA when the list of products fails to load.
 struct PointOfSaleItemListFullscreenErrorView: View {
     private let error: PointOfSaleErrorState
-    private let onRetry: (() -> Void)?
+    private let onAction: (() -> Void)?
 
-    init(error: PointOfSaleErrorState, onRetry: (() -> Void)? = nil) {
+    init(error: PointOfSaleErrorState, onAction: (() -> Void)? = nil) {
         self.error = error
-        self.onRetry = onRetry
+        self.onAction = onAction
     }
 
     var body: some View {
         PointOfSaleItemListFullscreenView {
-            PointOfSaleItemListErrorView(error: error, onRetry: onRetry)
+            PointOfSaleItemListErrorView(error: error, onAction: onAction)
         }
     }
 }
 
 #Preview {
-    PointOfSaleItemListFullscreenErrorView(error: .errorOnLoadingProducts(), onRetry: nil)
+    PointOfSaleItemListFullscreenErrorView(error: .errorOnLoadingProducts(), onAction: nil)
 }

WooCommerce/Classes/POS/Presentation/CardReaderConnection/UI States/PointOfSaleItemListFullscreenView.swift

@@ -31,6 +31,6 @@ private enum Localization {
     PointOfSaleItemListFullscreenView(
         content: {
             PointOfSaleItemListErrorView(
-                error: .init(title: "Error", subtitle: "Something went wrong", buttonText: "Fix it"))
+                error: .init(errorType: .productsLoadError, title: "Error", subtitle: "Something went wrong", buttonText: "Fix it"))
         })
 }

WooCommerce/Classes/POS/Presentation/CardReaderConnection/UI States/Reader Messages/PointOfSalePaymentSuccessView.swift

@@ -125,7 +125,7 @@ private extension PointOfSalePaymentSuccessView {
 #Preview {
     let posModel = PointOfSaleAggregateModel(
         itemsController: PointOfSalePreviewItemsController(),
-        couponsController: PointOfSalePreviewItemsController(),
+        couponsController: PointOfSalePreviewCouponsController(),
         cardPresentPaymentService: CardPresentPaymentPreviewService(),
         orderController: PointOfSalePreviewOrderController(),
         collectOrderPaymentAnalyticsTracker: POSCollectOrderPaymentAnalytics())

WooCommerce/Classes/POS/Presentation/CartView.swift

@@ -351,7 +351,7 @@ private extension CartView {
 #Preview {
     let posModel = PointOfSaleAggregateModel(
         itemsController: PointOfSalePreviewItemsController(),
-        couponsController: PointOfSalePreviewItemsController(),
+        couponsController: PointOfSalePreviewCouponsController(),
         cardPresentPaymentService: CardPresentPaymentPreviewService(),
         orderController: PointOfSalePreviewOrderController(),
         collectOrderPaymentAnalyticsTracker: POSCollectOrderPaymentAnalytics())
@@ -363,7 +363,7 @@ private extension CartView {
 #Preview("Cart with one item") {
     let posModel = PointOfSaleAggregateModel(
         itemsController: PointOfSalePreviewItemsController(),
-        couponsController: PointOfSalePreviewItemsController(),
+        couponsController: PointOfSalePreviewCouponsController(),
         cardPresentPaymentService: CardPresentPaymentPreviewService(),
         orderController: PointOfSalePreviewOrderController(),
         collectOrderPaymentAnalyticsTracker: POSCollectOrderPaymentAnalytics())

WooCommerce/Classes/POS/Presentation/Item Selector/ChildItemList.swift

@@ -89,7 +89,7 @@ private extension ChildItemList {
                 Spacer()
             }
 
-            PointOfSaleItemListErrorView(error: error, onRetry: {
+            PointOfSaleItemListErrorView(error: error, onAction: {
                 Task {
                     await posModel.loadItems(base: node)
                 }
@@ -152,7 +152,7 @@ private extension ChildItemList {
                 ], hasMoreItems: false)])
     let posModel = PointOfSaleAggregateModel(
         itemsController: itemsController,
-        couponsController: PointOfSalePreviewItemsController(),
+        couponsController: PointOfSalePreviewCouponsController(),
         cardPresentPaymentService: CardPresentPaymentPreviewService(),
         orderController: PointOfSalePreviewOrderController(),
         collectOrderPaymentAnalyticsTracker: POSCollectOrderPaymentAnalytics())
@@ -177,7 +177,7 @@ private extension ChildItemList {
         ])
     let posModel = PointOfSaleAggregateModel(
         itemsController: itemsController,
-        couponsController: PointOfSalePreviewItemsController(),
+        couponsController: PointOfSalePreviewCouponsController(),
         cardPresentPaymentService: CardPresentPaymentPreviewService(),
         orderController: PointOfSalePreviewOrderController(),
         collectOrderPaymentAnalyticsTracker: POSCollectOrderPaymentAnalytics())

WooCommerce/Classes/POS/Presentation/Item Selector/ItemList.swift

@@ -207,7 +207,7 @@ private extension ItemListRow {
 #Preview("Loading") {
     let posModel = PointOfSaleAggregateModel(
         itemsController: PointOfSalePreviewItemsController(),
-        couponsController: PointOfSalePreviewItemsController(),
+        couponsController: PointOfSalePreviewCouponsController(),
         cardPresentPaymentService: CardPresentPaymentPreviewService(),
         orderController: PointOfSalePreviewOrderController(),
         collectOrderPaymentAnalyticsTracker: POSCollectOrderPaymentAnalytics())