Merge pull request #60 from loudnate/stop-notifications

Clear "Loop Not Running" notifications when the number of configured …

Author: loudnate

Loop/Managers/DeviceDataManager.swift

@@ -109,6 +109,10 @@ class DeviceDataManager: CarbStoreDelegate, TransmitterDelegate {
         rileyLinkManager.disconnectDevice(device)
 
         AnalyticsManager.sharedManager.didChangeRileyLinkConnectionState()
+
+        if connectedPeripheralIDs.count == 0 {
+            NotificationManager.clearLoopNotRunningNotifications()
+        }
     }
 
     // MARK: Pump data

Loop/Managers/NotificationManager.swift

@@ -74,11 +74,21 @@ struct NotificationManager {
         UIApplication.sharedApplication().presentLocalNotificationNow(notification)
     }
 
+    // Cancel any previous scheduled notifications in the Loop Not Running category
+    static func clearLoopNotRunningNotifications() {
+        let app = UIApplication.sharedApplication()
+
+        app.scheduledLocalNotifications?.filter({
+            $0.category == Category.LoopNotRunning.rawValue
+        }).forEach({
+            app.cancelLocalNotification($0)
+        })
+    }
+
     static func scheduleLoopNotRunningNotifications() {
         let app = UIApplication.sharedApplication()
 
-        // Cancel any previous scheduled notifications
-        app.scheduledLocalNotifications?.filter({ $0.category == Category.LoopNotRunning.rawValue }).forEach({ app.cancelLocalNotification($0) })
+        clearLoopNotRunningNotifications()
 
         // Give a little extra time for a loop-in-progress to complete
         let gracePeriod = NSTimeInterval(minutes: 0.5)