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

dev next release 15.0.3 #8145

Open
wants to merge 68 commits into
base: master
Choose a base branch
from
Open

dev next release 15.0.3 #8145

wants to merge 68 commits into from

Conversation

bewest
Copy link
Member

@bewest bewest commented Oct 25, 2023

Please submit pull requests for next release here.

inventor96 and others added 30 commits October 16, 2019 14:18
…It displays "Internet offline." until internet is restored.

This fix addresses the issue that with a lost internet connection, clock view continues to display a stale CGM values forever, with no clue that internet connectivity is lost. This fix shows "Internet offline." for the duration that is it lost, then returns to normal when internet is restored.
Add "virtAsstLaunch"
Nightfoxy and others added 2 commits October 5, 2024 00:36
Changing default SAGE warn and urgent threshold defaults to be useful for 10 day sensors rather than 7 day sensors from the G5 era. SAGE pill turns yellow at day 9 and red 4 hours before expiration of 10 days.

Once G6 is retired, consider setting WARN to 10 days and URGENT to 10.5 days.
@bewest
Copy link
Member Author

bewest commented Nov 25, 2024

There are three high priority candidates for release:

There are some other PRs as well, help evaluating these as well as prepping the release and potential release notes are welcome.

AndyLow91 and others added 6 commits November 25, 2024 12:11
Updated to stop Nightscout crashing if APNs response is not received well from Apple.
…e' into wip/bewest/collaborations

Nice work, thanks for your contributions.
@AndyLow91
Copy link
Member

Let's chat about what's needed to push forward with dev @bewest 👍

This reverts commit cec3eeb.
Need to update tests?  Not immediately clear why tests updated from 6 days to 9
days fails, so revertinig for now.
@bewest
Copy link
Member Author

bewest commented Nov 25, 2024

@AndyLow91, sure thing. Please schedule some time with me on calendar, or find me on slack or discord so we can chat and coordinate properly.

@bewest
Copy link
Member Author

bewest commented Nov 29, 2024

@AndyLow91 I tried adding you to the reviewers list and it turns out I don't have admin access. 🤷

This fixes the issue of a "stale" CGM value displaying indefinately.
@bewest
Copy link
Member Author

bewest commented Dec 2, 2024

@AndyLow91, I think you should have received an invite to join a github team.
Also, this link joins the "Unified Diabetes Developer Alliance" is is informal in addition to the WeAreNotWaiting discord.
https://discord.gg/PpnRZfrj

Both areas contain discussions around the major areas that need development.

For this particular release, I was hoping to get some testing with AAPS on the mongo updates. It seems increasingly unlikely that any delay of this as a hotfix release will produce any useful information on the mongo testing. Is there any reason not to treat this as a hotfix and get it out by end of week?

The same question for mongo patches applies to nightscout-connect: is it better to scuttle potential release there for next release?
Thanks!

dsnallfot and others added 6 commits December 13, 2024 14:19
Switching the way units is fetched to this way makes it similar to the other
code already in place that does something similar.  This allows the tests to
pass with the existing fixtures.
Considering augmenting  additional tests that explicitly tests whether the
relevant branch of code is taken.
This adds the latest long term support version of node to the automated testing
matrix.
@bewest bewest requested a review from AndyLow91 December 14, 2024 20:39
@bewest bewest requested a review from TwistaTim December 17, 2024 22:56
Copy link
Member

@AndyLow91 AndyLow91 left a comment

Choose a reason for hiding this comment

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

All commits reviewed and I'm happy. It all makes sense. thanks

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.

9 participants