Merge pull request #6195 from woocommerce/issue/5980-wp3-manual

[Mobile Payments] Use configuration loader to decide which reader manual to show

Author: koke

WooCommerce/Classes/ViewRelated/Dashboard/Settings/In-Person Payments/InPersonPaymentsMenuViewController.swift

@@ -1,10 +1,13 @@
 import UIKit
 import SwiftUI
+import Yosemite
 
 final class InPersonPaymentsMenuViewController: UITableViewController {
     private var rows = [Row]()
+    private let configurationLoader: CardPresentConfigurationLoader
 
     init() {
+        configurationLoader = CardPresentConfigurationLoader()
         super.init(style: .grouped)
     }
 
@@ -28,10 +31,23 @@ private extension InPersonPaymentsMenuViewController {
     func configureRows() {
         rows = [
             .orderCardReader,
-            .manageCardReader,
-            .bbposChipper2XBTManual,
-            .stripeM2Manual
-        ]
+            .manageCardReader
+        ] + readerManualRows()
+    }
+
+    func readerManualRows() -> [Row] {
+        configurationLoader.configuration.supportedReaders.map { readerType in
+            switch readerType {
+            case .chipper:
+                return .bbposChipper2XBTManual
+            case .stripeM2:
+                return .stripeM2Manual
+            case .wisepad3:
+                return .wisepad3Manual
+            case .other:
+                preconditionFailure("Unknown card reader type was present in the supported readers list. This should not be possible")
+            }
+        }
     }
 
     func configureTableView() {
@@ -59,6 +75,8 @@ private extension InPersonPaymentsMenuViewController {
             configureChipper2XManual(cell: cell)
         case let cell as LeftImageTableViewCell where row == .stripeM2Manual:
             configureStripeM2Manual(cell: cell)
+        case let cell as LeftImageTableViewCell where row == .wisepad3Manual:
+            configureWisepad3Manual(cell: cell)
         default:
             fatalError()
         }
@@ -91,6 +109,13 @@ private extension InPersonPaymentsMenuViewController {
         cell.selectionStyle = .default
         cell.configure(image: .cardReaderManualIcon, text: Localization.stripeM2CardReaderManual)
     }
+
+    func configureWisepad3Manual(cell: LeftImageTableViewCell) {
+        cell.imageView?.tintColor = .text
+        cell.accessoryType = .disclosureIndicator
+        cell.selectionStyle = .default
+        cell.configure(image: .cardReaderManualIcon, text: Localization.wisepad3CardReaderManual)
+    }
 }
 
 // MARK: - Convenience methods
@@ -126,6 +151,10 @@ extension InPersonPaymentsMenuViewController {
     func stripeM2ManualWasPressed() {
         WebviewHelper.launch(Constants.stripeM2ManualURL, with: self)
     }
+
+    func wisepad3ManualWasPressed() {
+        WebviewHelper.launch(Constants.wisepad3ManualURL, with: self)
+    }
 }
 
 // MARK: - UITableViewDataSource
@@ -163,6 +192,8 @@ extension InPersonPaymentsMenuViewController {
             bbposChipper2XBTManualWasPressed()
         case .stripeM2Manual:
             stripeM2ManualWasPressed()
+        case .wisepad3Manual:
+            wisepad3ManualWasPressed()
         }
     }
 }
@@ -190,6 +221,11 @@ private extension InPersonPaymentsMenuViewController {
             "Stripe M2 card reader manual",
             comment: "Navigates to Stripe M2 Card Reader manual"
         )
+
+        static let wisepad3CardReaderManual = NSLocalizedString(
+            "WisePad 3 card reader manual",
+            comment: "Navigates to WisePad 3 Card Reader manual"
+        )
     }
 }
 
@@ -198,6 +234,7 @@ private enum Row: CaseIterable {
     case manageCardReader
     case bbposChipper2XBTManual
     case stripeM2Manual
+    case wisepad3Manual
 
     var type: UITableViewCell.Type {
         LeftImageTableViewCell.self
@@ -212,6 +249,7 @@ private enum Constants {
     static let woocommercePurchaseCardReaderURL = URL(string: "https://woocommerce.com/products/m2-card-reader/")!
     static let bbposChipper2XBTManualURL = URL(string: "https://developer.bbpos.com/quick_start_guide/Chipper%202X%20BT%20Quick%20Start%20Guide.pdf")!
     static let stripeM2ManualURL = URL(string: "https://stripe.com/files/docs/terminal/m2_product_sheet.pdf")!
+    static let wisepad3ManualURL = URL(string: "https://stripe.com/files/docs/terminal/wp3_product_sheet.pdf")!
 }
 
 // MARK: - SwiftUI compatibility

Yosemite/Yosemite/Model/Model.swift

@@ -133,6 +133,7 @@ public typealias CardReaderEvent = Hardware.CardReaderEvent
 public typealias CardReaderSoftwareUpdateState = Hardware.CardReaderSoftwareUpdateState
 public typealias CardReaderServiceDiscoveryStatus = Hardware.CardReaderServiceDiscoveryStatus
 public typealias CardReaderServiceError = Hardware.CardReaderServiceError
+public typealias CardReaderType = Hardware.CardReaderType
 public typealias CardReaderConfigError = Hardware.CardReaderConfigError
 public typealias PaymentParameters = Hardware.PaymentIntentParameters
 public typealias PaymentIntent = Hardware.PaymentIntent

Yosemite/Yosemite/Model/Payments/CardPresentPaymentsConfiguration.swift

@@ -4,11 +4,13 @@ public struct CardPresentPaymentsConfiguration {
     public let paymentMethods: [WCPayPaymentMethodType]
     public let currencies: [String]
     public let paymentGateways: [String]
+    public let supportedReaders: [CardReaderType]
 
-    init(paymentMethods: [WCPayPaymentMethodType], currencies: [String], paymentGateways: [String]) {
+    init(paymentMethods: [WCPayPaymentMethodType], currencies: [String], paymentGateways: [String], supportedReaders: [CardReaderType]) {
         self.paymentMethods = paymentMethods
         self.currencies = currencies
         self.paymentGateways = paymentGateways
+        self.supportedReaders = supportedReaders
     }
 
     public init(country: String, stripeEnabled: Bool, canadaEnabled: Bool) {
@@ -17,26 +19,29 @@ public struct CardPresentPaymentsConfiguration {
             self.init(
                 paymentMethods: [.cardPresent],
                 currencies: ["USD"],
-                paymentGateways: [WCPayAccount.gatewayID, StripeAccount.gatewayID]
+                paymentGateways: [WCPayAccount.gatewayID, StripeAccount.gatewayID],
+                supportedReaders: [.chipper, .stripeM2]
             )
         case "US" where stripeEnabled == false:
             self.init(
                 paymentMethods: [.cardPresent],
                 currencies: ["USD"],
-                paymentGateways: [WCPayAccount.gatewayID]
+                paymentGateways: [WCPayAccount.gatewayID],
+                supportedReaders: [.chipper, .stripeM2]
             )
         case "CA" where canadaEnabled == true:
             self.init(
                 paymentMethods: [.cardPresent, .interacPresent],
                 currencies: ["CAD"],
-                paymentGateways: [WCPayAccount.gatewayID]
+                paymentGateways: [WCPayAccount.gatewayID],
+                supportedReaders: [.wisepad3]
             )
         default:
-            self.init(paymentMethods: [], currencies: [], paymentGateways: [])
+            self.init(paymentMethods: [], currencies: [], paymentGateways: [], supportedReaders: [])
         }
     }
 
     public var isSupportedCountry: Bool {
-        paymentMethods.isEmpty == false && currencies.isEmpty == false && paymentGateways.isEmpty == false
+        paymentMethods.isEmpty == false && currencies.isEmpty == false && paymentGateways.isEmpty == false && supportedReaders.isEmpty == false
     }
 }