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

Push Current Value #67

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Push Current Value #67

wants to merge 2 commits into from

Conversation

kylerw
Copy link

@kylerw kylerw commented Jan 9, 2017

during each subscription update, push the current value of each subscribed device.

My ST development is nearly non-existent so there may be a more efficient way to do this but this is working for me.

during each subscription update, push the current value of each subscribed device.

My ST development is nearly non-existent so there may be a more efficient way to do this but this is working for me.
@kylerw
Copy link
Author

kylerw commented Jan 9, 2017

This should solve this #61 and #51

@kylerw
Copy link
Author

kylerw commented Jan 9, 2017

Doesn't poll the device but will simply send the current value/state. This helps for periodic reboots that might lose/forget states. Since updateSubscription() runs every 15 minutes there that would be the maximum amount of time that HomeAssistant is without states from ST.

@kylerw
Copy link
Author

kylerw commented Jan 9, 2017

Also: #66

@AceoStar
Copy link
Contributor

@kylerw Everytime I turn around I run into a thread where you're working on a commit that I need. We must have the exact same setup. :D

@stjohnjohnson
Copy link
Owner

I couldn't figure out how to give you a PR to your PR, so I just made a new one with the updated code.

However, it appears this may hit a rate limit issue with the device type.

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