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

Correction of state inversion in switches #21

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Correction of state inversion in switches #21

wants to merge 1 commit into from

Conversation

biau
Copy link

@biau biau commented Jun 12, 2018

I don't know why you've choose to flag state ON with 00 and OFF with 01 initially but this does not work with xiaomi wall switch connected to light (see https://community.home-assistant.io/t/new-custom-component-zigate/36820/282?u=biau )
With this changes wall switch works fine but I do not have aqara push button to see if this brake something.
I did not find any attribute value in the zigbee message to differentiate switches types.

@ISO-B
Copy link
Collaborator

ISO-B commented Jun 15, 2018

When switch is paired to zigate you could request manufacturer info. Easier solution might be to add configuration parameter for inverting ON and OFF states. Other good solution could be adding manufacturer and model parameters to config and using them on code.

I don't have aquara push button either, but https://zigate.fr/xiaomi-bouton-switch-cluster/ shows that 00 is ON and 01 is OFF.

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.

None yet

2 participants