Merge pull request #5133 from woocommerce/issue/5131-low-batt-disable-update-button

Author: allendav

WooCommerce/Classes/ViewRelated/Dashboard/Settings/CardReadersV2/CardReaderSettingsConnectedViewController.swift

@@ -70,6 +70,7 @@ private extension CardReaderSettingsConnectedViewController {
         guard let viewModel = viewModel else {
             return
         }
+
         viewModel.checkForCardReaderUpdate()
     }
 
@@ -199,7 +200,13 @@ private extension CardReaderSettingsConnectedViewController {
         }
 
         if readerUpdateAvailable == .isTrue {
-            cell.configure(image: .infoOutlineImage, text: Localization.updateAvailable)
+            let readerBatteryTooLow = viewModel?.readerBatteryTooLowForUpdates ?? false
+
+            if readerBatteryTooLow {
+                cell.configure(image: .infoOutlineImage, text: Localization.updateAvailableLowBatt)
+            } else {
+                cell.configure(image: .infoOutlineImage, text: Localization.updateAvailable)
+            }
             cell.backgroundColor = .warningBackground
             cell.imageView?.tintColor = .warning
         }
@@ -231,7 +238,8 @@ private extension CardReaderSettingsConnectedViewController {
 
         let readerDisconnectInProgress = viewModel?.readerDisconnectInProgress ?? false
         let readerUpdateInProgress = viewModel?.readerUpdateInProgress ?? false
-        cell.enableButton(readerUpdateAvailable && !readerDisconnectInProgress && !readerUpdateInProgress)
+        let readerBatteryTooLow = viewModel?.readerBatteryTooLowForUpdates ?? false
+        cell.enableButton(readerUpdateAvailable && !readerDisconnectInProgress && !readerUpdateInProgress && !readerBatteryTooLow)
         cell.showActivityIndicator(readerUpdateInProgress)
 
         cell.selectionStyle = .none
@@ -378,6 +386,11 @@ private extension CardReaderSettingsConnectedViewController {
             comment: "Settings > Manage Card Reader > Connected Reader > A prompt to update a reader running older software"
         )
 
+        static let updateAvailableLowBatt = NSLocalizedString(
+            "An update is available, but your reader battery is too low to update. Please charge your reader right away to continue accepting payments",
+            comment: "Settings > Manage Card Reader > Connected Reader > A prompt to charge a reader running older software"
+        )
+
         static let updateNotNeeded = NSLocalizedString(
             "Congratulations! Your reader is running the latest software",
             comment: "Settings > Manage Card Reader > Connected Reader > A prompt to update a reader running older software"

WooCommerce/Classes/ViewRelated/Dashboard/Settings/CardReadersV2/CardReaderSettingsConnectedViewModel.swift

@@ -12,6 +12,7 @@ final class CardReaderSettingsConnectedViewModel: CardReaderSettingsPresentedVie
 
     private(set) var checkForReaderUpdateInProgress: Bool = false
     private(set) var readerUpdateAvailable: CardReaderSettingsTriState = .isUnknown
+    private(set) var readerBatteryTooLowForUpdates: Bool = false
     private(set) var readerUpdateInProgress: Bool = false
     private(set) var readerUpdateCompletedSuccessfully: Bool = false
 
@@ -61,6 +62,8 @@ final class CardReaderSettingsConnectedViewModel: CardReaderSettingsPresentedVie
             return
         }
 
+        readerBatteryTooLowForUpdates = batteryLevel < Constants.batteryLevelNeededForUpdates
+
         let batteryLevelPercent = Int(100 * batteryLevel)
         let batteryLevelString = NumberFormatter.localizedString(from: batteryLevelPercent as NSNumber, number: .decimal)
         connectedReaderBatteryLevel = String.localizedStringWithFormat(Localization.batteryLabelFormat, batteryLevelString)
@@ -181,6 +184,14 @@ final class CardReaderSettingsConnectedViewModel: CardReaderSettingsPresentedVie
     }
 }
 
+// MARK: - Constants
+//
+private extension CardReaderSettingsConnectedViewModel {
+    enum Constants {
+        static let batteryLevelNeededForUpdates = Float(0.5)
+    }
+}
+
 // MARK: - Localization
 //
 private extension CardReaderSettingsConnectedViewModel {