Skip to content
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

Update client.js #13

Closed
wants to merge 1 commit into from
Closed

Update client.js #13

wants to merge 1 commit into from

Conversation

aaronjknox
Copy link

Set High Dashed line to 230, Set Low Dashed line to 90

Set High Dashed line to 230, Set Low Dashed line to 90
@jasoncalabrese
Copy link
Member

@aaronjknox We need to make this configurable instead of just changing the hardcoded values. I think your threshold will be considered too high for most people.

@aaronjknox
Copy link
Author

Hi Jason, I agree, please disregard/delete my request. I got into making
changes thinking I was in my personal monitor page. Just trying to learn as
I go.
Thanks for helping to create the cloud monitor.
On Jun 18, 2014 9:13 PM, "Jason Calabrese" notifications@github.com wrote:

@aaronjknox https://github.com/aaronjknox We need to make this
configurable instead of just changing the hardcoded value. I think your
threshold will be considered too high for most people.


Reply to this email directly or view it on GitHub
#13 (comment)
.

@aaronjknox aaronjknox deleted the Threshold-shift branch June 21, 2014 00:31
jasoncalabrese pushed a commit that referenced this pull request Nov 29, 2014
MilosKozak pushed a commit that referenced this pull request Jun 6, 2018
mrspouse pushed a commit to mrspouse/cgm-remote-monitor that referenced this pull request Apr 3, 2022
hydroxy pushed a commit to hydroxy/cgm-remote-monitor that referenced this pull request Dec 3, 2022
DarynGit added a commit to DarynGit/cgm-remote-monitor that referenced this pull request Jun 20, 2024
bewest pushed a commit that referenced this pull request Dec 16, 2024
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.

2 participants