Add niko-home-control documentation#5137
Add niko-home-control documentation#5137fabaff merged 2 commits intohome-assistant:nextfrom NoUseFreak:feature/niko-home-control
Conversation
| host: 192.168.1.123 | ||
| ``` | ||
|
|
||
| Configuration variables: |
There was a problem hiding this comment.
Please see https://www.home-assistant.io/developers/documentation/standards/ for configuration variables.
| --- | ||
|
|
||
| The `niko_home_control` platform allows you to integrate your [Niko Home Control](https://www.niko.eu/enus/products/niko-home-control) into Home Assistant. | ||
|
|
There was a problem hiding this comment.
It would be nice if you would describe less briefly what types of devices are supported / what a potential user of the component/platform can expect to have with this platform.
|
|
||
| Configuration variables: | ||
|
|
||
| - **host** (*Required*): The ip address your Niko Home is on, |
There was a problem hiding this comment.
Is this the only parameter, or was something left out?
There was a problem hiding this comment.
I assume this might be a hostname as well?
Secondly, ip needs to be capitalized; IP.
|
|
||
| Configuration variables: | ||
|
|
||
| - **host** (*Required*): The ip address your Niko Home is on, |
There was a problem hiding this comment.
I assume this might be a hostname as well?
Secondly, ip needs to be capitalized; IP.
| footer: true | ||
| ha_category: Light | ||
| ha_iot_class: "Local Polling" | ||
| ha_release: 0.36 |
There was a problem hiding this comment.
Incorrect release number, 0.36 is pretty old. The suggestion is to update it to 0.68.
|
@NoUseFreak Thank you for this PR. I and @rytilahti reviewed the PR and left you some comments. Another suggestion, maybe, add the logo of Niko to the documentation as well. |
frenck
added
the
new-integration
|
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
stale
Bot
added
the
stale
|
Reopen for home-assistant/core#18019. |
stale
Bot
removed
the
stale
frenck
added
next
4 participants
Description:
Pull request in home-assistant (if applicable): home-assistant/core#13755
Checklist:
current. New documentation for platforms/components and features should go tonext.