reservoir change analytics, readme update (#105)

Author: loudnate

Loop/Managers/AnalyticsManager.swift

@@ -75,6 +75,10 @@ class AnalyticsManager {
         logEvent("Pump battery replacement")
     }
 
+    func reservoirWasRewound() {
+        logEvent("Pump reservoir rewind")
+    }
+
     func didChangeBasalRateSchedule() {
         logEvent("Basal rate change")
     }

Loop/Managers/DeviceDataManager.swift

@@ -281,6 +281,10 @@ class DeviceDataManager: CarbStoreDelegate, TransmitterDelegate, ReceiverDelegat
                         NotificationManager.sendPumpReservoirLowNotificationForAmount(newVolume, andTimeRemaining: timeLeft)
                     }
                 }
+
+                if newVolume > previousVolume + 1 {
+                    AnalyticsManager.sharedManager.reservoirWasRewound()
+                }
             }
         }
     }

README.md

@@ -30,7 +30,7 @@ Please understand that this project:
   <thead>
     <tr>
       <td colspan="2" rowspan="4"></td>
-      <th colspan="2">Insulin Pump</th>
+      <th colspan="3">Insulin Pump</th>
     </tr>
     <tr>
       <th>MM 522/722</th>
@@ -43,27 +43,28 @@ Please understand that this project:
       <th rowspan="4">CGM</th>
       <th>Dexcom G4</th>
       <td>✅<sup><a href="#hw1">1</a> <a href="#hw2">2</a></sup></td>
-      <td>✅<sup><a href="#hw1">1</a></sup></td>
+      <td>✅<sup><a href="#hw1">1</a> <a href="#hw3">3</a></sup></sup></td>
+      <td>✅<sup><a href="#hw1">1</a> <a href="#hw3">3</a></sup></sup></td>
     </tr>
     <tr>
       <th>Dexcom G5</th>
-      <td>✅<sup><a href="#hw2">2</a> <a href="#hw3">3</a></sup></td>
+      <td>✅<sup><a href="#hw2">2</a></sup></td>
+      <td>✅<sup><a href="#hw3">3</a></sup></td>
       <td>✅<sup><a href="#hw3">3</a></sup></td>
     </tr>
     <tr>
       <th>MM CGM</th>
       <td>❌<sup><a href="#hw4">4</a></sup></td>
-      <td>✅</td>
-      <td>✅<sup><a href="#hw5">5</a></sup></td>
+      <td>✅<sup><a href="#hw3">3</a></sup></td>
+      <td>✅<sup><a href="#hw3">3</a></sup></td>
     </tr>
   </tbody>
 </table>
 
 <br/><a name="hw1">1</a>. Offline access to glucose requires a Receiver with Share and the [Share2 app](https://itunes.apple.com/us/app/dexcom-share2/id834775275?mt=8) to be running on the same device. Internet-dependent access via Share servers is also supported.
 <br/><a name="hw2">2</a>. Pump must have a remote ID added in the [Remote Options](https://www.medtronicdiabetes.com/sites/default/files/library/download-library/workbooks/x22_menu_map.pdf) menu.
-<br/><a name="hw3">3</a>. Early firmware only available on select models is required for using Closed Loop and Bolus features.
+<br/><a name="hw3">3</a>. Early firmware (US <= 2.4A, AU/EUR <= 2.6A) is required for using Closed Loop and Bolus features.
 <br/><a name="hw4">4</a>. It's not impossible, but comms-heavy and there's some work to be done. File an issue if you're someone who's up for the challenge and can test this hardware configuration.
-<br/><a name="hw4">5</a>. Confirmed working with AU MM554 firmware 2.6A - unsuccessful reports from some, need to check individual pumps.
 
 ### RileyLink
 
@@ -94,9 +95,9 @@ Loop optionally supports select third-party remote services, which are configure
 | Service                | Description
 | ---------------------- | -------------
 | Dexcom Share           | Downloads glucose data if a local G5 Transmitter or G4 Receiver with Share is not available.
-| Nightscout             | Uploads treatments and other pump data. Note you will need to set "Nightscout history uploading" to "On" in Settings for treatments to be fetched from your pump and uploaded to Nightscout.
+| Nightscout             | Uploads treatments and other pump data. Note that you will need to set "Nightscout history uploading" to "On" in Settings for treatments to be fetched from your pump and uploaded to Nightscout.
 | mLab                   | Uploads diagnostic data about each loop run, as well as app errors. At this time, it is strongly recommended that you configure this service in case retrospective analysis is needed.
-| Amplitude              | Tracks private, single-user behavioral and system analytics
+| Amplitude              | Tracks private, single-user behavioral and system analytics (no health data is sent)
 
 # Making it Your Own