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

Clarify z-wave polling. #8291

Merged
merged 4 commits into from Feb 2, 2019
Merged

Clarify z-wave polling. #8291

merged 4 commits into from Feb 2, 2019

Conversation

sveip
Copy link
Contributor

@sveip sveip commented Jan 26, 2019

Description:

Pull request in home-assistant (if applicable): home-assistant/home-assistant#

Checklist:

  • Branch: next is for changes and new documentation that will go public with the next home-assistant release. Fixes, changes and adjustments for the current release should be created against current.
  • The documentation follows the standards.

@ghost ghost added the to-do label Jan 26, 2019
@klaasnicolaas klaasnicolaas added enhancement ready-for-review This PR needs to be reviewed current This PR goes into the current branch and removed to-do labels Jan 26, 2019
typo

Co-Authored-By: sveip <petersveistrup@gmail.com>
klaasnicolaas
klaasnicolaas previously approved these changes Jan 26, 2019
DubhAd
DubhAd previously requested changes Jan 26, 2019
source/_docs/z-wave/devices.markdown Outdated Show resolved Hide resolved
@klaasnicolaas
Copy link
Member

I have adjusted the last line, please give your opinion.

I miss for the lack of clarity where / how you can turn on the polling, if I understand correctly you do that by means of the polling intensity, 0 = off and > 0 is on?

@turbokongen
Copy link
Contributor

Polling is disabled if not specified in config. If Polling is changed in the ui, 0 is off anything else is on.

@DubhAd DubhAd dismissed their stale review February 2, 2019 11:29

There's been changes

Copy link
Contributor

@DubhAd DubhAd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@DubhAd DubhAd merged commit 3625d60 into home-assistant:current Feb 2, 2019
@ghost ghost removed the ready-for-review This PR needs to be reviewed label Feb 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
current This PR goes into the current branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants