Updated aREST switch doc for invert#5377
Conversation
| - **name** (*Optional*): Let you overwrite the name of the device. By default *name* from the device is used. | ||
| - **pins** array (*Optional*): An array with all used pins. | ||
| - **name** (*Required*): The name of the pin to use in the frontend. | ||
| - **invert** (*Optional*): If the logic for on/off should be inverted. 1 = off and 0 = on |
There was a problem hiding this comment.
Please add the default value. The type (bool) would be nice, too. A {% configuration %} block would be awesome. :-)
There was a problem hiding this comment.
@w1ll1am23 @syssi
1 = off? So true, is not to invert?
There was a problem hiding this comment.
No no, I was trying to call out a reference to the aREST code. (Arduino) 1 or HIGH is off and 0 or LOW is on.
There was a problem hiding this comment.
Aah check
@w1ll1am23, are you planning on changing/updating this PR, based on the feedback by @syssi ?
There was a problem hiding this comment.
@frenck yes, didn't get a chance yesterday. I'll try to get it done later today.
fabaff
added
the
new-feature
| type: string | ||
| invert: | ||
| description: If the logic for on/off should be inverted. | ||
| required: optional |
There was a problem hiding this comment.
required: false. Is optional also valid?
There was a problem hiding this comment.
Well valid as in "Does it work?"; yes. Matching our standards: no.
* Updated aREST switch doc for invert * Updated to configuration block * ✏️ Changed values of required configuration keys * ✏️ Language tweak
4 participants
Description:
Pull request in home-assistant (if applicable): home-assistant/core#14467
Checklist:
current. New documentation for platforms/components and features should go tonext.