-
-
Notifications
You must be signed in to change notification settings - Fork 7.7k
Added documentation for flic smart button component. #1527
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
Conversation
Fixed capital letters of Home Assistant.
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.
Looks good with one small comment.
Also fixed a small formatting error.
Looks good in general and can be merged after this is fixed and parent pr is merged.
|
||
Configuration variables: | ||
|
||
- **host** (*Optional*): The IP of the flic service server. (default: `localhost`) |
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.
Should be "IP or hostname". Example even includes the hostname of localhost.
@Landrash I updated the documentation. |
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.
Looks great and can be merged after parent is merged. 🍸
Can you include an example automation making use of the new events? See https://home-assistant.io/components/lirc/ for an example. |
@armills Done 😉 |
Renamed ‘auto_scan’ to ‘discovery’.
…1527) * Added documentation for flic component.
@fabianbergmark I created the new PR #1556 for this. |
Description:
Added documentation for flic smart button component.
Pull request in home-assistant (if applicable): home-assistant/core#4681