-
Notifications
You must be signed in to change notification settings - Fork 514
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
Remove actualDate
as introduced in #22
#55
Conversation
Proposed in conjunction with nightscout/trio-oref#14
Accidentally created the PR from my fork, but the same branch is available at https://github.com/nightscout/Open-iAPS/tree/remove_actualDate |
I pushed in the branch remove_actualDate some fix : see f5d301e Idid some tests described here using the corrected version of oref :
|
- allows to add carbs in past or future - allows to add FPU only use createdAt attribute
the initial branch submodule+oref contains some modified code not included in this PR/branch. I create a new PR : #56 |
@avouspierre , I guess this is ready to be merged into dev? |
👍 |
Thanks! Merging this with two approvals. |
Merge pull request #55 from bjornoleh/remove_actualDate
Proposed in conjunction with nightscout/trio-oref#14