-
Notifications
You must be signed in to change notification settings - Fork 166
OmniKit wip branch. #380
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
Merged
OmniKit wip branch. #380
Conversation
This file contains hidden or 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
ec05c2f to
c84fb48
Compare
Seperated Bolus tests from Message tests.
Added beepType options to cancel and configure alerts command
* Setup steps and ways to contribute with Omnipod development * Reduced images sizes screenshots * Aligned screenshots and changed title headers * Added more steps before creating a PR * removed testflight + howto signup developper account * Removed NS options and fixed typo
* 'Fixed missing Foundation and availability of Bolus tests in Xcode' * Updated statusTypes * added status test and fixed typo * removed obsolete unknown requesttype function * added more statusFaultError tests and values * Added more basic test values to StatusError Type2 * Added to statusError: - insulinRemaining - timeActive - secondaryLoggedFaultEvent - logEventError - New test for testing one Fault during deactivation * Added remaining values: - insulinStateCorruptionDuringErrorLogging - progressAtFirstLoggedFaultEvent - recieverLowGain - radioRSSI - progressAtFirstLoggedFaultEventCheck - Fixed wrong low bit selection on radio Removed StatusConfigAlarms test for now, must use separate route to evaluate. * * Enum values in lowerCase * DeliveryType as OptionSet * Fix Typo original * Added extendedBolus value to OptionSet DeliveryType, to be the same as StatusError DeliveryType. * Messageblock error: Function not closed with } * Fixed typo original * Added .none DeliveryType * * Removed DeliveryInProgress error message handling * Added none type * Fixed typo original * Added extra pod failure tests * The statuslist of StatusError and StatusResponse is the same, so: * Put all ReservoirStatus cases into StatusResponse * Replaced ProgressType with ReservoirStatus in StatusError
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.
Requires 2.0 versions subg_rfspy and ble_rfspy on your RileyLink.