nate/fix/LOOP-1927/manual-glucose-entry (#15)

nhamming · web-flow · commit a888409b7815 · 2020-09-22T18:01:44.000-03:00
* updates to align with renaming in LoopKit

* removed glucose range categorization

* clean up
diff --git a/ShareClient/ShareClientManager.swift b/ShareClient/ShareClientManager.swift
@@ -61,7 +61,7 @@ public class ShareClientManager: CGMManager {
 
     public let shouldSyncToRemoteService = false
 
-    public var sensorState: SensorDisplayable? {
+    public var glucoseDisplay: GlucoseDisplayable? {
         return latestBackfill
     }
 
diff --git a/ShareClient/ShareGlucose+GlucoseKit.swift b/ShareClient/ShareGlucose+GlucoseKit.swift
@@ -22,7 +22,7 @@ extension ShareGlucose: GlucoseValue {
 }
 
 
-extension ShareGlucose: SensorDisplayable {
+extension ShareGlucose: GlucoseDisplayable {
     public var isStateValid: Bool {
         return glucose >= 39
     }
@@ -36,12 +36,12 @@ extension ShareGlucose: SensorDisplayable {
     }
     
     // TODO Placeholder. This functionality will come with LOOP-1311
-    public var glucoseValueType: GlucoseValueType? {
+    public var glucoseRangeCategory: GlucoseRangeCategory? {
         return nil
     }
 }
 
-extension SensorDisplayable {
+extension GlucoseDisplayable {
     public var stateDescription: String {
         if isStateValid {
             return LocalizedString("OK", comment: "Sensor state description for the valid state")

Original file line number	Diff line number	Diff line change
`@@ -61,7 +61,7 @@ public class ShareClientManager: CGMManager {`
`61`	`61`
`62`	`62`	`public let shouldSyncToRemoteService = false`
`63`	`63`
`64`		`- public var sensorState: SensorDisplayable? {`
	`64`	`+ public var glucoseDisplay: GlucoseDisplayable? {`
`65`	`65`	`return latestBackfill`
`66`	`66`	`}`
`67`	`67`
Original file line number	Diff line number	Diff line change
`@@ -22,7 +22,7 @@ extension ShareGlucose: GlucoseValue {`
`22`	`22`	`}`
`23`	`23`
`24`	`24`
`25`		`-extension ShareGlucose: SensorDisplayable {`
	`25`	`+extension ShareGlucose: GlucoseDisplayable {`
`26`	`26`	`public var isStateValid: Bool {`
`27`	`27`	`return glucose >= 39`
`28`	`28`	`}`
`@@ -36,12 +36,12 @@ extension ShareGlucose: SensorDisplayable {`
`36`	`36`	`}`
`37`	`37`
`38`	`38`	`// TODO Placeholder. This functionality will come with LOOP-1311`
`39`		`- public var glucoseValueType: GlucoseValueType? {`
	`39`	`+ public var glucoseRangeCategory: GlucoseRangeCategory? {`
`40`	`40`	`return nil`
`41`	`41`	`}`
`42`	`42`	`}`
`43`	`43`
`44`		`-extension SensorDisplayable {`
	`44`	`+extension GlucoseDisplayable {`
`45`	`45`	`public var stateDescription: String {`
`46`	`46`	`if isStateValid {`
`47`	`47`	`return LocalizedString("OK", comment: "Sensor state description for the valid state")`