-
-
Notifications
You must be signed in to change notification settings - Fork 7.7k
Update z-wave.markdown #1452
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
Update z-wave.markdown #1452
Conversation
@turbokongen, thanks for your PR! By analyzing the history of the files in this pull request, we identified @balloob, @Landrash and @devdelay to be potential reviewers. |
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.
One missing word and one question. Always lovely to see your work on the zwave stuff.
@@ -79,6 +79,8 @@ Configuration variables: | |||
- **customize** (*Optional*): This attribute contains node-specific override values: | |||
- **polling_intensity** (*Optional*): Enables polling of a value and sets the frequency of polling (0=none, 1=every time through the list, 2=every other time, etc). If not specified then your device will not be polled. | |||
- **ignored** (*Optional*): Ignore this entitiy completely. It won't be shown in the Web Interface and no events are generated for it. | |||
- **refresh_value** (*Optional*): Enable refreshing of node value. Only light component uses this. Defaults to 2 second delay. |
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.
Missing a "the". Looks good with that exception.
@@ -79,6 +79,8 @@ Configuration variables: | |||
- **customize** (*Optional*): This attribute contains node-specific override values: | |||
- **polling_intensity** (*Optional*): Enables polling of a value and sets the frequency of polling (0=none, 1=every time through the list, 2=every other time, etc). If not specified then your device will not be polled. | |||
- **ignored** (*Optional*): Ignore this entitiy completely. It won't be shown in the Web Interface and no events are generated for it. | |||
- **refresh_value** (*Optional*): Enable refreshing of node value. Only light component uses this. Defaults to 2 second delay. | |||
- **delay** (*Optional*): Specify the delay to wait for refresh of node value if you want other than 2 seconds. |
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.
Plural of values from node or should it be a single value?
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.
It's this specific value for this node.
Description:
Pull request in home-assistant (if applicable): home-assistant/core#4437