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

Support Z2M for Legrand controllers #224

Merged
merged 5 commits into from
Jan 23, 2021
Merged

Support Z2M for Legrand controllers #224

merged 5 commits into from
Jan 23, 2021

Conversation

Crocmagnon
Copy link
Contributor

⚠️ I've not tested it locally but I'm confident since these are the states published on MQTT and found in the *_action sensor 😇

@Crocmagnon
Copy link
Contributor Author

Also I guess the docs need to be updated but I'm not familiar with your data structure. Please tell me if you want me to invest some more time in updating it!

@xaviml
Copy link
Owner

xaviml commented Jan 20, 2021

Hi @Crocmagnon,

Thank you for your contribution. If you want, you can add the Z2M mapping for both controller in the documentation:

You can take a look for an example in this one: https://github.com/xaviml/controllerx/blob/dev/docs/_data/controllers/E1743.yml

You just need to add Zigbee2MQTT in the integrations list. Order is:

  • z2m
  • deconz
  • zha

If you don't add it, I will add it in your branch when I get time before merging the PR.

Regards,
Xavi M.

@Crocmagnon
Copy link
Contributor Author

Thanks for the examples 👍 I updated the docs to reflect the Z2M addition and took the liberty of fixing the existing docs to match the actual behavior of the buttons and mapped actions.

@xaviml xaviml merged commit baf28ad into xaviml:dev Jan 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants