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

Ia7 #685

Merged
merged 2 commits into from
Apr 4, 2017
Merged

Ia7 #685

merged 2 commits into from
Apr 4, 2017

Conversation

hplato
Copy link
Collaborator

@hplato hplato commented Mar 30, 2017

Two fixes, an undefined error if a subpage is loaded without going to the IA7 root first. And a fix for the issue Dean Junk brought up "LONG_POLL when clicking on browse items & categories"

@hplato
Copy link
Collaborator Author

hplato commented Apr 4, 2017

Dean tested this so I'm going to merge into master

@hplato hplato merged commit 6b705b8 into hollie:master Apr 4, 2017
@hplato
Copy link
Collaborator Author

hplato commented Apr 13, 2017

Try updating from master to see if 1.4 fixed it, and if you use weather station 2, it will refresh data every minute (custom refresh rate can be set in ia7_config.json).

It doesn't use a long_poll, since reading the RRD multiple times a second put a large load on my development environment. Since by default the weather RRDs are updated every minute, a timed 60 second refresh minimizes the demands on the client and server.

@tobser
Copy link
Contributor

tobser commented Apr 13, 2017

I just pulled master and so far everything is working.

I do not use weather station, so I can't comment on that. But thanks for adding the other fix so quickly.

@hplato
Copy link
Collaborator Author

hplato commented Apr 13, 2017 via email

@tobser
Copy link
Contributor

tobser commented Apr 13, 2017

Yep those RRD graphs look like a fun thing to have.. I really should free up some time to set those up...

@peloy
Copy link
Collaborator

peloy commented Apr 13, 2017 via email

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.

3 participants