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

Add external insulin #104

Closed
wants to merge 13 commits into from

Conversation

avouspierre
Copy link
Contributor

@avouspierre avouspierre commented Apr 14, 2024

The PR includes the #93 and #80 to add external insulin in the interface.
The PR includes #101
The PR updates tidepool to add / delete external insulin and iSMB logic

dataview add view
Simulator Screenshot - iPhone 15 - 2024-04-14 at 11 43 24 Simulator Screenshot - iPhone 15 - 2024-04-14 at 11 43 45

BrianWieder and others added 9 commits April 13, 2024 20:32
Co-authored-by: dnzxy <d.c.cengiz@googlemail.com>
Co-Authored-By: Deniz Cengiz <48965855+dnzxy@users.noreply.github.com>
Co-Authored-By: Deniz Cengiz <48965855+dnzxy@users.noreply.github.com>
Update tidepool to add external insulin
Co-Authored-By: Deniz Cengiz <48965855+dnzxy@users.noreply.github.com>
@avouspierre
Copy link
Contributor Author

resolve issue #77 with isSMB attribute available.

BrianWieder and others added 4 commits April 15, 2024 20:53
NightscoutTreatment should be agnostic, so moving the init extension into PumpHistoryStorage allows for better sepearation of concerns.
Update tidepool after removing externalInsulin event type by @BrianWieder
@marionbarker
Copy link
Contributor

Is this superceded by PR #140?

@MikePlante1 MikePlante1 deleted the branch nightscout:dev May 4, 2024 13:14
@MikePlante1 MikePlante1 closed this May 4, 2024
@MikePlante1 MikePlante1 reopened this May 4, 2024
@MikePlante1 MikePlante1 changed the base branch from alpha to dev May 4, 2024 13:49
@MikePlante1
Copy link
Contributor

@avouspierre Sorry, I accidentally closed this when I deleted the alpha branch. I reopened it and changed it to target dev now instead.

@MikePlante1 MikePlante1 changed the title [alpha] - Add external insulin in alpha Add external insulin May 4, 2024
@marionbarker
Copy link
Contributor

I believe this PR was superceded by PR #140.
@MikePlante1 It is possible that git is smarter than you when it automatically closed this.

@AndreasStokholm
Copy link
Contributor

@marionbarker Whenever a branch is deleted, GitHub automatically closes all PRs targeting that branch.

@dnzxy
Copy link
Contributor

dnzxy commented May 11, 2024

Can this be closed as #140 introduces all its contents and supercedes it?

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

Successfully merging this pull request may close these issues.

7 participants