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

[NEW DEVICE] Zigbee2Mqtt seems to have changed Xiaomi WXKG07LM actions. #584

Closed
SergioRius opened this issue Sep 30, 2022 · 3 comments
Closed
Assignees
Labels
new-device New device

Comments

@SergioRius
Copy link

New Device Support

Device Information

  • Device Model: WXKG07LM
  • Device Description: Xiaomi 2 rocker battery wall switch
  • Device Manufacturer: Xiaomi

Integrations

Integration: z2m

Actions

Before, actions where:
right, right_double, right_long
left, left_double, left_long
....

Now have changed to:
single_right, double_right, hold_right
single_left, double_left, hold_left
single_both, double_both, hold_both

Notes

The device page in the Z2M database seems to ha recently been modified and reflects the new actions. I can't fetch the old contents, though.
imagen

I've noticed this because I had to re-pair all the devices after a gateway change. Before, they where paired using the old actions, but after the re-pair, those Aqara switches didn't work anymore so I debugged the actions published and after changing them for the controllerx apps that do merge_mapping, they worked.
But default mappings should be changed in code. I don't feel confident enough to do a pull request on this as I'm unsure where to find all the related code. Only found the one relative to the last ZM change.

@SergioRius SergioRius added the new-device New device label Sep 30, 2022
@SergioRius SergioRius changed the title [NEW DEVICE] [OLD DEVICE] Zigbee2Mqtt seems to have changed Xiaomi WXKG07LM actions. [NEW DEVICE] Zigbee2Mqtt seems to have changed Xiaomi WXKG07LM actions. Sep 30, 2022
xaviml added a commit that referenced this issue Oct 3, 2022
@xaviml
Copy link
Owner

xaviml commented Oct 3, 2022

Hi @SergioRius ,

Thank you for opening this issue :) I just created and merged this PR with the fix: #587

I will be releasing the fix soon, but in the meantime, you can map those actions with mapping or merge_mapping as you mentioned.

Regards,
Xavi M.

@SergioRius
Copy link
Author

Your welcome. I'm so sorry for not requesting a PR myself.
I've already installed the latest beta version and checked that those changes work properly. Thank you.

@xaviml
Copy link
Owner

xaviml commented Oct 8, 2022

Hi @SergioRius ,

This device mapping is now fixed in ControllerX v4.24.0.

Regards,
Xavi M.

@xaviml xaviml closed this as completed Oct 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new-device New device
Projects
None yet
Development

No branches or pull requests

2 participants