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 state every 30 minutes #71

Closed
wants to merge 1 commit into from
Closed

Conversation

stjohnjohnson
Copy link
Owner

This will push the current state of all attributes every 30 minutes, so if the bridge was down, it will eventually recover to current state.

Should replace #67 and resolve #61, #51, and #66.

This will push the current state of all attributes every 30 minutes, so if the bridge was down, it will eventually recover to current state.
@stjohnjohnson
Copy link
Owner Author

Eh, this actually isn't a good plan of attack. I'm getting rate limited at the device handler level. :(

http://docs.smartthings.com/en/latest/ratelimits/index.html?highlight=rate#ways-to-avoid-hitting-smartapp-and-device-handler-rate-limits

@kylerw
Copy link

kylerw commented Feb 2, 2017

Maybe less frequently but also on demand?

@stjohnjohnson
Copy link
Owner Author

stjohnjohnson commented Feb 2, 2017 via email

@Michael-Bell
Copy link

Any progress with the rate limits? Is it possible to use something like runin() to go through every device in smaller groups over time, then wait until it's been X minutes since the first device in the list was updated and repeat?

@alex-w-k
Copy link

Any updates on this? I'd really like my battery information to be sent over and this PR should fix it.

@stjohnjohnson
Copy link
Owner Author

I've almost moved completely off SmartThings (API was too limiting and it's entirely cloud based still). So I'm not really working on this right now. You're welcome to take the code I've put here and fiddle with it yourself. Honestly, the best route is to have some sort of bulk API to receive an array of events.

@stjohnjohnson
Copy link
Owner Author

Closing as there isn't really a clear plan forward. I'll leave the branch around but SmartThings has some hard limits.

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.

Battery not sent (ever?)
4 participants