-
Notifications
You must be signed in to change notification settings - Fork 71.8k
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
Modified Index.html to enable 2 hour alarm silence #21
Conversation
I cherry-picked the useful bits here: brianhanifin#5 When that PR goes through, I'll close this one. I'm considering adding the config file to the |
Greg, if you can manage to: git checkout master
git checkout 75d1c52d4441
git checkout -b restore-original-01
git push origin restore-original-01 This will create a branch I can target for upgrade on your fork. Afterwards, you can delete/rename your old master branch to something else, and call this one your new master. Most people don't add their own commits on top of master since that creates trouble in pulling changes from elsewhere. Happy to explain in more detail. |
Conflicts: static/index.html
Hey Ben - I can definitely do that, but one question -- when I look at Nightscout master repo, line 31, what I modified is already merged. So I think it's all done. https://github.com/nightscout/cgm-remote-monitor/blob/master/static/index.html#L31 Two questions though:
Thanks!
From: Ben West [mailto:notifications@github.com] Greg, if you can manage to: git checkout master This will create a branch I can target for upgrade on your fork. Afterwards, you can delete/rename your old master branch to something else, and call this one your new master. Most people don't add their own commits on top of master since that creates trouble in pulling changes from elsewhere. Happy to explain in more detail. — |
Closing this, the relevant commit was cherry-picked and is in master now. |
CSS Fix: time + glucose collision
Replace \n with a new line, as heroku program does not export it well.
Added another time interval to index.html to align with Dexcom 2 hour window. Our insulin action is 2 hours and that's what we use for "at night." Plus, with longer silence, it will reduce website traffic.