-
Notifications
You must be signed in to change notification settings - Fork 171
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
Release 1.0.1 #327
Merged
Release 1.0.1 #327
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Owner
ps2
commented
Apr 21, 2017
•
edited
Loading
edited
- Error handling improvements. Splitting bolus errors into certain and uncertain failure categories.
- Add pump timezone, bolusing, and suspended states to pump status in ns upload
- Split Suspend/Resume into separate NS event types
- Adding notes fields to treatments.
- Check pump status for suspend and bolusing states before bolusing. And allow clearing of current temp basal before bolusing.
* Tests for Events & PumpOpsSynchronous * clarify some comments, remove unneeded and potentionally confusing tests * Specify destination for xcodebuild by uuid * Update simulator to 10.2 * update xcode to 8.2 as well
* Fix warnings for swift3.1 * Don't call description directly * Simplify conversion of port attribute to Int
Build CI on Xcode8.3
pump fields
sync dev with all from master
Bump version 1.0.1
Revert PumpOpsSynchronous changes until we are finished with tests
* Check pump status for suspend and bolusing states before bolusing. And allow clearing of current temp basal before bolusing. * Back out testing code, and signing * failures to clear temp basal before bolus should be certain errors * Handle PumpCommandError when clearing temp for bolus as certain error, move SetBolusError to PumpOpsSynchronous
Include pump date in response to request for reservoir level
* Update documentation for bolus command * document default
Expose BolusNightscoutTreatment as public
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.