add warning to multi-point calibration option

jdtoppin · Jan 28, 2023 · 7eb822c · 7eb822c
1 parent 7bd4f6b
commit 7eb822c
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 1 deletion.
diff --git a/xdrip/Storyboards/en.lproj/BluetoothPeripheralView.strings b/xdrip/Storyboards/en.lproj/BluetoothPeripheralView.strings
@@ -40,4 +40,4 @@
 "nfcScanSuccessfulMessage" = "\n✅ Scan successful ✅\n\nClick OK and just wait to allow the sensor to finish connecting via bluetooth.\n\nIn a short while the status should change to 'Connected'.";
 "nfcErrorMessageScanErrorRetrying" = "Sensor scan error\n\nRetrying... # ";
 "nfcErrorMessageScanFailed" = "Sensor scan has failed";
-
+"nonFixedSlopeWarning" = "Multi-point calibration is an advanced feature.\n\nPlease do not use this feature until you have read the calibration section of the online help and understand how it works.";
diff --git a/xdrip/Texts/TextsBluetoothPeripheralView.swift b/xdrip/Texts/TextsBluetoothPeripheralView.swift
@@ -140,4 +140,8 @@ class Texts_BluetoothPeripheralView {
         return NSLocalizedString("nfcScanNeeded", tableName: filename, bundle: Bundle.main, value: "NFC scan needed", comment: "text in status row, when waiting for a successful NFC scan before starting bluetooth scanning")
     }()
 
+    static let nonFixedSlopeWarning: String = {
+        return NSLocalizedString("nonFixedSlopeWarning", tableName: filename, bundle: Bundle.main, value: "Multi-point calibration is an advanced feature.\n\nPlease do not use this feature until you have read the calibration section of the online help and understand how it works.", comment: "text to inform the user that multi-point calibration is an advanced option and could be dangerous if used incorrectly")
+    }()
+
 }
diff --git a/...sViewController/BluetoothPeripheralViewController/BluetoothPeripheralViewController.swift b/...sViewController/BluetoothPeripheralViewController/BluetoothPeripheralViewController.swift
@@ -1350,6 +1350,13 @@ extension BluetoothPeripheralViewController: UITableViewDataSource, UITableViewD
 
                     self.bluetoothPeripheral?.blePeripheral.nonFixedSlopeEnabled = isOn
 
+                    // show the user a warning if they enabled multi-point calibration as this should only be used by advanced users who understand how it works. Too many new users were enabling this and getting bad results.
+                    if isOn {
+
+                        self.present(UIAlertController(title: Texts_Common.warning, message: Texts_BluetoothPeripheralView.nonFixedSlopeWarning, actionHandler: nil), animated: true, completion: nil)
+
+                    }
+
                     // send info to bluetoothPeripheralManager
                     if let bluetoothPeripheral = self.bluetoothPeripheral {