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

feat(sensors): Support for new Dexcom One+ sensors #6

Merged
merged 3 commits into from
Mar 9, 2024

Conversation

Sjoerd-Bo3
Copy link
Contributor

@Sjoerd-Bo3 Sjoerd-Bo3 commented Mar 8, 2024

Connect using G7 manager in Open-iAPS

Credit: Marek Lelicki

Connect using G7 manager in Open-iAPS
@Sjoerd-Bo3 Sjoerd-Bo3 changed the base branch from main to dev March 8, 2024 10:14
@Sjoerd-Bo3 Sjoerd-Bo3 changed the title Support for new Dexcom One+ sensors feat(sensors): Support for new Dexcom One+ sensors Mar 8, 2024
@Sjoerd-Bo3 Sjoerd-Bo3 changed the title feat(sensors): Support for new Dexcom One+ sensors feat(sensors): Support for the new Dexcom One+ sensors Mar 8, 2024
@Sjoerd-Bo3 Sjoerd-Bo3 changed the title feat(sensors): Support for the new Dexcom One+ sensors feat(sensors): Support for new Dexcom One+ sensors Mar 8, 2024
@Sjoerd-Bo3
Copy link
Contributor Author

testing webhook

@Sjoerd-Bo3 Sjoerd-Bo3 self-assigned this Mar 8, 2024
@bjornoleh
Copy link
Contributor

@Sjoerd-Bo3 looks good, but there’s a merge conflict in the md. Will approve when that’s sorted 😀

@MikePlante1
Copy link
Contributor

While I assume this fix works because it's the same code that was committed to iAPS, does anyone have access to a ONE+ to actually test it?

@MikePlante1
Copy link
Contributor

@Sjoerd-Bo3 looks good, but there’s a merge conflict in the md. Will approve when that’s sorted 😀

Sorted.

@bjornoleh
Copy link
Contributor

While I assume this fix works because it's the same code that was committed to iAPS, does anyone have access to a ONE+ to actually test it?

I think this can be merged without actual testing, given the reports from Loop and iAPS.

@MikePlante1 MikePlante1 self-requested a review March 8, 2024 16:42
Copy link
Contributor

@lifeguardz lifeguardz left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

@Sjoerd-Bo3
Copy link
Contributor Author

@Sjoerd-Bo3 looks good, but there’s a merge conflict in the md. Will approve when that’s sorted 😀

Sorted.

Yeah I will merge dev myself next time. Forgot😂

@Sjoerd-Bo3 Sjoerd-Bo3 merged commit 1a2ebf0 into dev Mar 9, 2024
@Sjoerd-Bo3 Sjoerd-Bo3 deleted the feat-dexcomoneplus branch March 9, 2024 07:06
@MikePlante1 MikePlante1 mentioned this pull request Mar 9, 2024
bjornoleh referenced this pull request in bjornoleh/Trio May 13, 2024
mountrcg pushed a commit to mountrcg/Trio that referenced this pull request Aug 1, 2024
Improve gradient colors (thanks @tqb43), removed padding
mountrcg pushed a commit to mountrcg/Trio that referenced this pull request Aug 1, 2024
…cout#9 & nightscout#18 and MinimedKit PR nightscout#6 (nightscout#438)

+ Extend low reservoir notification range
+ Prevent negative duration doses, which can happen if the phone clock is changed
+ Fixes Loop crash when changing time back and cancelling bolus, Loop Issue #2057
mountrcg pushed a commit to mountrcg/Trio that referenced this pull request Aug 1, 2024
mountrcg pushed a commit to mountrcg/Trio that referenced this pull request Aug 1, 2024
…block-fetching

Add block NS fetch toggle functionality from Trio
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.

5 participants