Skip to content
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

Add a better way to integrate ZHA devices #97

Closed
xaviml opened this issue Jul 9, 2020 · 1 comment
Closed

Add a better way to integrate ZHA devices #97

xaviml opened this issue Jul 9, 2020 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@xaviml
Copy link
Owner

xaviml commented Jul 9, 2020

Feature Request

Is your feature request related to a problem?

Until now, adding new devices for any integration was as easy as creating a mapping between the fired action of the controller and the action that the event should do. For the case of z2m is easy since all events are string specifying the action that the controller is taking. For deCONZ is more or less the same but with event ID, except the Xiaomi cube where the events are taen from the gesture keyword. However for ZHA is completely different.

In case of ZHA the events fire a JSON with a command (string) and argument (it can be anything). There is no standard established in how the devices should fire events, this is why it depends on the device in specific.

Describe the solution / feature you'd like

What I am aiming in this issue is to be able to easily integrate new ZHA devices by implementing a specific function for each device rather than making it generic as it is until now.

Additional context

This will allow us to integrate the Xiaomi cube for ZHA and the Xiaomi Mijia Smart Switch Round ( #94 ). Until this is not tackled, those devices cannot be yet integrated.

@xaviml xaviml added the enhancement New feature or request label Jul 9, 2020
@xaviml xaviml self-assigned this Jul 9, 2020
xaviml added a commit that referenced this issue Jul 11, 2020
@xaviml
Copy link
Owner Author

xaviml commented Jul 12, 2020

The integration for this device has been added in ControllerX v3.2.0.

@xaviml xaviml closed this as completed Jul 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant