This is the Tidepool commit history for trio branch between previous commit and commit proposed by this PR.
b286256 fix merge conflict mistake
600e685 Merge branch 'dev' into trio
6e007cb Merge pull request #22 from LoopKit/manual-dose-upload
e914eec Upload manual doses as normal bolus for now
a2ccad7 Merge pull request #1 from marionbarker/non-pump-insulin
Details:
The previous commit used by Trio for TidepoolService used LoopKit/TidepoolService PR 21 for fixing the upload of non-pump insulin to Tidepool
This initial version (PR 21) was merged into the trio branch while the PR was still open at LoopKit
A different PR was created and merged into LoopKit/TidepoolService:dev
The code in PR 22 is slightly different but the effect is the same
This PR means the TidepoolService submodule used by Trio is identical to that used by Loop
The gist of both PR 21 and 22 was to correct a problem where non-pump (Loop) or external (Trio) insulin doses were not uploaded to Tidepool
Note that there is an open Issue 77 that is still open and is not affected by this PR
All boluses are still labeled as automatic in the Tidepool website display
Comparison with Loop
I confirmed that for Loop:
all automatic boluses are labeled as automatic in Tidepool and displayed with a darker blue
all boluses manually delivered by Loop or entered as non-pump insulin are unlabeled (therefore manual) and displayed as a lighter blue by Tidepool
Status:
Tested and it works.
Description:
This PR is an Index change for the TidepoolService submodule:
This is the Tidepool commit history for trio branch between previous commit and commit proposed by this PR.
Details:
Comparison with Loop
I confirmed that for Loop: