-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Only allow one active glucose data source #364
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Separate CGM sources out into their own section and enforce zero or one active sources at any given time. This requires a new boolean value to track whether the G5 transmitter is enabled independently of the actual transmitter ID. While we're at it, create a separate config section for the pump to group it with the pump battery type since those are not really configuration values and go together.
|
Some notes:
|
The top animation is a clearer indication that this is an extension of the G5 Transmitter enabled switch.
| static let count = 2 | ||
| } | ||
|
|
||
| fileprivate enum CgmRow: Int { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/Cgm/CGM
dynamic dispatch. Also make the internal methods private for good measure.
|
pull request am i |
|
This looks good to me. Are there parts of the guide that should be updated? |
|
I can do this. Just updating Anna's app now. This went into dev branch though...so I will note that in guide and when it goes to master I'll update the guide for that too. |

Separate CGM sources out into their own section and enforce zero or one
active sources at any given time. This requires a new boolean value to
track whether the G5 transmitter is enabled independently of the actual
transmitter ID.
While we're at it, create a separate config section for the pump to
group it with the pump battery type since those are not really
configuration values and go together.