Skip to content

Commit 37c7525

Browse files
authored
Red loop retry should refresh device data (#1381)
1 parent bef58e5 commit 37c7525

File tree

2 files changed

+22
-7
lines changed

2 files changed

+22
-7
lines changed

Loop/Managers/DeviceDataManager.swift

Lines changed: 21 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -419,16 +419,31 @@ extension DeviceDataManager: PumpManagerDelegate {
419419
}
420420
lastBLEDrivenUpdate = Date()
421421

422-
cgmManager?.fetchNewDataIfNeeded { (result) in
423-
if case .newData = result {
424-
AnalyticsManager.shared.didFetchNewCGMData()
425-
}
422+
refreshCGM()
423+
}
424+
425+
private func refreshCGM(_ completion: (() -> Void)? = nil) {
426+
if let cgmManager = cgmManager {
427+
cgmManager.fetchNewDataIfNeeded { (result) in
428+
if case .newData = result {
429+
AnalyticsManager.shared.didFetchNewCGMData()
430+
}
426431

427-
if let manager = self.cgmManager {
428432
self.queue.async {
429-
self.processCGMResult(manager, result: result)
433+
self.processCGMResult(cgmManager, result: result)
434+
completion?()
430435
}
431436
}
437+
} else {
438+
completion?()
439+
}
440+
}
441+
442+
func refreshDeviceData() {
443+
refreshCGM() {
444+
self.queue.async {
445+
self.pumpManager?.assertCurrentPumpData()
446+
}
432447
}
433448
}
434449

Loop/View Controllers/StatusTableViewController.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1282,7 +1282,7 @@ final class StatusTableViewController: ChartsTableViewController {
12821282
if error != nil {
12831283
let alertController = UIAlertController(with: error!)
12841284
let manualLoopAction = UIAlertAction(title: NSLocalizedString("Retry", comment: "The button text for attempting a manual loop"), style: .default, handler: { _ in
1285-
self.deviceManager.loopManager.loop()
1285+
self.deviceManager.refreshDeviceData()
12861286
})
12871287
alertController.addAction(manualLoopAction)
12881288
present(alertController, animated: true)

0 commit comments

Comments
 (0)