Skip to content

Conversation

@ps2
Copy link
Collaborator

@ps2 ps2 commented Dec 14, 2019

Pause launch if launching before first unlock. Workaround for #1170

@marionbarker
Copy link
Contributor

I just tested this on my phone. I used Katie D's low power trick to make my phone mess up, then built the new version of the code and repeated the trick and Loop worked nicely. Dexcom has added a squawk that alerts you their app isn't working properly. Had to force quit Dexcom app both reboots. Loop opened nicely with all settings after the fix.
iPhone 11 pro
iOS 13.2.3
gitBranch: settings-loss-patch
Omnipod

Thanks Pete.

@ps2
Copy link
Collaborator Author

ps2 commented Dec 16, 2019

Thanks Marion. Looking for other testers to try this out. Not as much to check that it fixes the issue, as I believe that is working, but to watch out for regressions.

@skimaniac
Copy link

skimaniac commented Dec 16, 2019

I tested it and everything looks ok now with the fix.
I tried with the cell unlocked and forced the start (power button and low volume) and also with the cell locked and forced also the start, if I remember correctly this is how I had lost my settings at least once.

My only concerns is that it takes 2 to 3 mins before a normal graph is displayed, after starting I had a prediction of 0 mmol/L in 1 hour and -16 mmol/L in about 4 h , whereas it is 6.3 in 1 hour and 7 in 4 hours.

IMG_0975

IMG_0973

Thank you!

Iphone 7 plus
IOS 13.3

@ps2 ps2 merged commit d22b837 into dev Dec 19, 2019
@ps2 ps2 deleted the settings-loss-patch branch December 19, 2019 03:41
novalegra pushed a commit to novalegra/Loop that referenced this pull request Jan 31, 2020
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.

4 participants