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

NOT ISSUE - New Aqara accessory #1360

Closed
GerSant opened this issue Apr 1, 2019 · 11 comments
Closed

NOT ISSUE - New Aqara accessory #1360

GerSant opened this issue Apr 1, 2019 · 11 comments

Comments

@GerSant
Copy link

GerSant commented Apr 1, 2019

Only for you information

Aqara has shipped a new relay module LLKZMK11LM

https://homekitnews.com/2019/03/27/aqara-wireless-relay-controller/

Regards

@lolorc
Copy link
Contributor

lolorc commented Apr 1, 2019

just bought one :-)

@fmolero
Copy link
Contributor

fmolero commented Apr 1, 2019

I think I receive it this week

@Koenkk
Copy link
Owner

Koenkk commented Apr 1, 2019

It's even already supported in the dev branch 😄

@Koenkk Koenkk closed this as completed Apr 1, 2019
@fmolero
Copy link
Contributor

fmolero commented Apr 9, 2019

Hi, I am testing this device with zigbee2mqtt and everything works fine except that the publication in mqtt does not work to trigger a relay or another. In my case, if I publish

zigbee2mqtt/[Device ID]/set {"state":"ON"} or {"state":"OFF"} works fine, but only activates or deactivates the L1 relay.

If I publish in zigbee2mqtt/[Device ID]/l1 or l2/set {"state":"ON"} or {"state":"OFF"} I receive the following message

abr 09 16:11:31 JeedomServer npm[12775]: zigbee2mqtt:error 2019-4-9 16:11:31 Failed to find device with ieeAddr: 'XiaomiSwitch1/l1'
abr 09 16:11:31 JeedomServer npm[12775]: zigbee2mqtt:info 2019-4-9 16:11:31 MQTT publish: topic 'zigbee2mqtt/bridge/log', payload '{"type":"entity_not_found","message":"XiaomiSwitch1/l1"}'
abr 09 16:11:31 JeedomServer npm[12775]: zigbee2mqtt:warn 2019-4-9 16:11:31 Cannot handle MQTT message on 'zigbee2mqtt/XiaomiSwitch1/l1/set' with data '{"state":"OFF"}'

How can I activate a relay or another?

@fmolero
Copy link
Contributor

fmolero commented Apr 11, 2019

After investigating, I have noticed that the definition of this product is not well done at all and there is a small bug. The endpoints are defined in device.js as l1 and l2 but nevertheless if we use zigbee2mqtt / DEVICE_ID / l1 / set it does not work, however if it works if we use zigbee2mqtt / DEVICE_ID / left / set, for example or right. So I have changed in device.js l1 and l2 by left and right and it is already working correctly. Should we define l1 and l2 somewhere else, apart from device.js?

@Koenkk
Copy link
Owner

Koenkk commented Apr 11, 2019

@fmolero yes, l1 and l2 need to be added to this list: https://github.com/Koenkk/zigbee2mqtt/blob/master/lib/util/utils.js#L31 can you make a pr?

@fmolero
Copy link
Contributor

fmolero commented Apr 11, 2019

@Koenkk , Do I do the pr in the master or dev branch?

@Koenkk
Copy link
Owner

Koenkk commented Apr 11, 2019

Dev branch.

@fmolero
Copy link
Contributor

fmolero commented Apr 11, 2019

Ok, done.

@Koenkk
Copy link
Owner

Koenkk commented Apr 11, 2019

Can you provide a link to the pr? I dont see it.

@fmolero
Copy link
Contributor

fmolero commented Apr 12, 2019

Sorry, something had not done well. I just created the pr again, here is the link.
#1411

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

No branches or pull requests

4 participants