-
Notifications
You must be signed in to change notification settings - Fork 6
Feature/custom buttons remote #12
Feature/custom buttons remote #12
Conversation
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.
Thanks for your contribution!
Please add a pull request description, what this changes are for. Especially for the new sendCustomCommand
method.
…s to addAvailableEntity method with empty string list as default
Hi, could you take another look at the two commits i added. I think supporting the custom features is possible with minimal changes to the addAvailableEntity method. (Added customFeatures string list parameter with empty list as default) |
@nikos1988 I'll have a look tomorrow. I didn't see the 3rd part of your PR, maybe I just missed it. Shouldn't there be an implementation in an integration plugin, e.g. home-assistant, openhab or homey? |
Hi, the 3rd PR was for the web-configurator to add the custom features. I have developed an mqtt integration, but this is currently customized for my own needs. Maybe i can release something better suited for everyone in the future. |
Maybe we can join our efforts: I'm working on a webhook integration which would be very easy to extend with mqtt: |
Since there hasn't been any activity for a while and the PR looks ready, I'm going to change the state to ready and review it. |
PR 2/3