-
-
Notifications
You must be signed in to change notification settings - Fork 28.5k
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
verisure configurable polling #17144
verisure configurable polling #17144
Conversation
documentation for home-assistant/core#17144
@@ -53,6 +54,8 @@ | |||
vol.Optional(CONF_SMARTPLUGS, default=True): cv.boolean, | |||
vol.Optional(CONF_THERMOMETERS, default=True): cv.boolean, | |||
vol.Optional(CONF_SMARTCAM, default=True): cv.boolean, | |||
vol.Optional(CONF_POLLING_RATE, default=1): vol.All( | |||
vol.Coerce(int), vol.Range(min=1)), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We usually use cv.time_period
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That would allow the user to set seconds, right? Verisure seems to lock people out that poll too often so I rather see that a minute is minimum.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can set a minimum time too.
@@ -53,6 +54,8 @@ | |||
vol.Optional(CONF_SMARTPLUGS, default=True): cv.boolean, | |||
vol.Optional(CONF_THERMOMETERS, default=True): cv.boolean, | |||
vol.Optional(CONF_SMARTCAM, default=True): cv.boolean, | |||
vol.Optional(CONF_POLLING_RATE, default=1): vol.All( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Normally we call this CONF_SCAN_INTERVAL
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok. I should change that. When is the next release?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In ~7 days.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great., I will be able to update during the weekend. Thanks @MartinHjelmare
Description:
Some users have received mail from Verisure informing them to poll less.
Related issue (if applicable): fixes #
Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#<home-assistant.io PR number goes here>
Example entry for
configuration.yaml
(if applicable):Checklist:
tox
. Your PR cannot be merged unless tests passIf user exposed functionality or configuration variables are added/changed:
If
[ex-requir]: https://github.com/home-assistant/home-assistant/blob/dev/homeassistant/components/keyboard.py#L14
[ex-import]: https://github.com/home-assistant/home-assistant/blob/dev/homeassistant/components/keyboard.py#L54