StephenBlackWasAlreadyTaken / xDrip

Android Application that collects dex signals, allows calibrations, and uploads
GNU General Public License v3.0
208 stars 176 forks source link

Wixel Battery Display! #70

Open syntaxerr66 opened 9 years ago

syntaxerr66 commented 9 years ago

TODO:

syntaxerr66 commented 9 years ago

I added a try-catch for exception handling of in case the 2nd element of the array wasn't transmitted, and it'll show it on the home screen that it was missing.

StephenBlackWasAlreadyTaken commented 9 years ago

also thoughts on using the same shades as the rest of the app (orangeish and dark redish and white in place of green?) I forget the exact values but we should probably constantize them somewhere, you can find them in the BgGraphBuilder class

syntaxerr66 commented 9 years ago

Well crap, I didn't realize my experimental stuff would get shoved into the PR. The popup and colors and stuff weren't supposed to be part of the PR. Didn't know that if I push to my master, github automagically incorporates it. I learned something new today.

I'll make the changes you noted (I meant to do notification, but was tired and wanted to just make something work), and the colors.

StephenBlackWasAlreadyTaken commented 9 years ago

Looks good! 2 minor issues still, we need the migration that added the battery level column (you can roll them both into migration 21 because presumably no one currently running this has one and not the other?) and we have some merge conflicts again