Skip to content
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

Insulin Delivery #463

Merged
merged 3 commits into from Jan 4, 2023
Merged

Conversation

bassrock
Copy link
Contributor

@bassrock bassrock commented Jan 2, 2023

This PR adds Insulin Delivery to the Application and also uploads it to Apple Health and Nightscout.

I also updated the Health integration to delete glucose and insulin records from Apple Health when they originated from Glucose Direct and they are deleted within Glucose Direct.

I chose to use the insulin options that NightScout uses (Meal Bolus, Correction Bolus, Snack Bolus) and then I downgrade them for use in Apple Health which only supports Bolus and Basal.

Some of these requests are in #214

NavigationView {
HStack {
Form {
Section(footer: Text("Basal insulin referes to the insulin used to regulate blood glucose between meals including during sleep. Bolus insulin refers to the insulin used to regulate blood glucose at meals and/or to acutely address high blood glucose.")) {
Copy link
Contributor Author

@bassrock bassrock Jan 2, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy that is similar to Apple Health manual data logging. Could remove if wanted.

@creepymonster creepymonster added the enhancement New feature or request label Jan 3, 2023
@creepymonster
Copy link
Owner

I really like the sheet for entering the insulin. I will definitely adopt this for entering blood glucose values/calibrations.

@bassrock
Copy link
Contributor Author

bassrock commented Jan 3, 2023

@creepymonster thanks! Are there any steps you'd like me to take to get this good for merging?

@creepymonster
Copy link
Owner

No, I think it's great that way. I will do the merge and in the next step take over the sheet for glucose values/calibrations.

A topic that will then be interesting is the display of insulin in the chart (but no longer the compatibility chart for iOS 15 devices).

@creepymonster creepymonster merged commit e3e913c into creepymonster:main Jan 4, 2023
@creepymonster
Copy link
Owner

Thank you very much, I was very excited about the PR yesterday.

I myself have always shied away from it, because I myself do not enter carbohydrates ☺️. I have always entered them in FAX/Loop.

@bassrock
Copy link
Contributor Author

bassrock commented Jan 4, 2023

@creepymonster yep! I was thinking about displaying it on the chart as well in a future pr, similar to how the libre app does it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants