Update ineligible UI when feature switch is disabled to enable & retry.

Author: jaclync

WooCommerce/Classes/POS/TabBar/POSIneligibleView.swift

@@ -51,7 +51,7 @@ struct POSIneligibleView: View {
                             }
                         }
                     } label: {
-                        Text(Localization.refreshEligibility)
+                        Text(reason.refreshEligibilityTitle)
                     }
                     .buttonStyle(POSFilledButtonStyle(size: .normal, isLoading: isLoading))
                     .renderedIf(reason.shouldShowRetryButton)
@@ -91,9 +91,9 @@ struct POSIneligibleView: View {
                                      value: "Install and activate the WooCommerce plugin from your WordPress admin.",
                                      comment: "Suggestion for missing WooCommerce plugin: install plugin")
         case .featureSwitchDisabled:
-            return NSLocalizedString("pos.ineligible.suggestion.featureSwitchDisabled",
+            return NSLocalizedString("pos.ineligible.suggestion.featureSwitchDisabled.2",
                                      value: "Point of Sale must be enabled to proceed. " +
-                                     "Please enable the POS feature from your WordPress admin under WooCommerce settings > Advanced > Features.",
+                                     "You can enable the POS feature below or from your WordPress admin under WooCommerce settings > Advanced > Features.",
                                      comment: "Suggestion for disabled feature switch: enable feature in WooCommerce settings")
         case let .unsupportedCurrency(supportedCurrencies):
             let currencyList = supportedCurrencies.map { $0.rawValue }
@@ -127,12 +127,6 @@ private extension POSIneligibleView {
             comment: "Title shown in POS ineligible view"
         )
 
-        static let refreshEligibility = NSLocalizedString(
-            "pos.ineligible.refresh.button.title",
-            value: "Retry",
-            comment: "Button title to refresh POS eligibility check"
-        )
-
         static let dismiss = NSLocalizedString(
             "pos.ineligible.dismiss.button.title",
             value: "Exit POS",
@@ -156,6 +150,28 @@ private extension POSIneligibleReason {
             return true
         }
     }
+
+    var refreshEligibilityTitle: String {
+        switch self {
+        case .featureSwitchDisabled:
+            return NSLocalizedString(
+                "pos.ineligible.enable.pos.feature.and.refresh.button.title",
+                value: "Enable POS & Retry",
+                comment: "Button title to enable the POS feature switch and refresh POS eligibility check"
+            )
+        case .unsupportedIOSVersion,
+                .unsupportedWooCommerceVersion,
+                .siteSettingsNotAvailable,
+                .wooCommercePluginNotFound,
+                .unsupportedCurrency,
+                .selfDeallocated:
+            return NSLocalizedString(
+                "pos.ineligible.refresh.button.title",
+                value: "Retry",
+                comment: "Button title to refresh POS eligibility check"
+            )
+        }
+    }
 }
 
 #if DEBUG

WooCommerce/Classes/ViewRelated/Dashboard/Settings/POS/POSTabEligibilityChecker.swift

@@ -13,6 +13,9 @@ import enum Yosemite.FeatureFlagAction
 import enum Yosemite.SettingAction
 import protocol Yosemite.POSSystemStatusServiceProtocol
 import class Yosemite.POSSystemStatusService
+import protocol Yosemite.POSSiteSettingServiceProtocol
+import class Yosemite.POSSiteSettingService
+import enum Networking.SiteSettingsFeature
 
 /// Represents the reasons why a site may be ineligible for POS.
 enum POSIneligibleReason: Equatable {
@@ -50,6 +53,7 @@ final class POSTabEligibilityChecker: POSEntryPointEligibilityCheckerProtocol {
     private let stores: StoresManager
     private let featureFlagService: FeatureFlagService
     private let systemStatusService: POSSystemStatusServiceProtocol
+    private let siteSettingService: POSSiteSettingServiceProtocol
 
     init(siteID: Int64,
          userInterfaceIdiom: UIUserInterfaceIdiom = UIDevice.current.userInterfaceIdiom,
@@ -58,14 +62,16 @@ final class POSTabEligibilityChecker: POSEntryPointEligibilityCheckerProtocol {
          stores: StoresManager = ServiceLocator.stores,
          featureFlagService: FeatureFlagService = ServiceLocator.featureFlagService,
          systemStatusService: POSSystemStatusServiceProtocol = POSSystemStatusService(credentials: ServiceLocator.stores.sessionManager.defaultCredentials,
-                                                                                      storageManager: ServiceLocator.storageManager)) {
+                                                                                      storageManager: ServiceLocator.storageManager),
+         siteSettingService: POSSiteSettingServiceProtocol = POSSiteSettingService(credentials: ServiceLocator.stores.sessionManager.defaultCredentials)) {
         self.siteID = siteID
         self.userInterfaceIdiom = userInterfaceIdiom
         self.siteSettings = siteSettings
         self.eligibilityService = eligibilityService
         self.stores = stores
         self.featureFlagService = featureFlagService
         self.systemStatusService = systemStatusService
+        self.siteSettingService = siteSettingService
     }
 
     /// Checks the initial visibility of the POS tab without dependance on network requests.
@@ -133,8 +139,7 @@ final class POSTabEligibilityChecker: POSEntryPointEligibilityCheckerProtocol {
         case .unsupportedWooCommerceVersion, .wooCommercePluginNotFound:
             return await checkEligibility()
         case .featureSwitchDisabled:
-            // TODO: WOOMOB-759 - enable feature switch via API and check eligibility again
-            // For now, just checks eligibility again.
+            _ = try await siteSettingService.setFeature(siteID: siteID, feature: .pointOfSale, enabled: true)
             return await checkEligibility()
         case .selfDeallocated:
             return await checkEligibility()