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

Fixed Missing Device Selector for xiaomi_wxkg11lm Zigbee2MQTT to include Aqara. #690

Merged
merged 5 commits into from
Feb 16, 2025

Conversation

yarafie
Copy link
Contributor

@yarafie yarafie commented Feb 15, 2025

Thank you for taking the time to work on a Pull Request. Your contribution is really appreciated! 🎉
Please don't delete any part of the template, since keeping the provided structure will help maintainers to review your work more rapidly.

Sections marked as * are required and need to be filled in.

Proposed change*

Fixed Missing Device Selector for Zigbee2MQTT to include Aqara.
Addresses:
Fixes #686

Checklist*

  • [Y] I followed sections of the Contribution Guidelines relevant to changes I'm proposing.
  • [Y] I properly tested proposed changes on my system and confirm that they are working as expected.
  • [Y] I formatted files with Prettier using the command npm run format before submitting my Pull Request.

Copy link
Contributor

github-actions bot commented Feb 15, 2025

Hey @EPMatt, thank you so much for your contribution! 🚀

🔄 We're currently running a few checks to make sure that everything is great with your contribution.
If further actions need to be performed before your contribution can be reviewed, additional guidance will be provided to you in the next comment.

Results are coming soon, stay tuned!

@yarafie yarafie changed the title Fixed Missing Device Selector for Zigbee2MQTT to include Aqara. Fixed Missing Device Selector for xiaomi_wxkg11lm Zigbee2MQTT to include Aqara. Feb 16, 2025
Copy link
Owner

@EPMatt EPMatt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thank you for the quick fix @yarafie.

Before merging this:

  • Let's remember to also bump version number to 2025-02-16
  • I did a quick research. It looks like the Xiaomi WXKG01LM is a different device than the Aqara WXKG11LM. Hence, I believe we should add an entry to the Controllers docs with a separate image. No need to duplicate the actual blueprint as long as the interface of the two device is exactly the same.

No need to do anything on your side, I can take care of this later today.

@EPMatt EPMatt merged commit ecfda7a into EPMatt:main Feb 16, 2025
4 of 5 checks passed
github-actions bot pushed a commit that referenced this pull request Feb 16, 2025
…aomi_wxkg11lm Zigbee2MQTT to include Aqara. (#690)

* Fixed Missing Device Selector for Zigbee2MQTT to include Aqara.

* docs: add separate entry for aqara and xiaomi controllers

* feat: separate Aqara WXKG11LM Wireless Mini Switch / Xiaomi WXKG01LM Mi Wireless Switch in hooks, change controller blueprint name

* docs: bump hook version to 2025.02.16, update changelogs

* npm run format

---------

Co-authored-by: EPMatt <[email protected]>
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.

Bug - Aqara Wireless Switch Mini No Longer Found
2 participants