-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Add compatibility for 3-Groups RCU from EGLO #5853
Conversation
@Koenkk this is the requested PR for Koenkk/zigbee2mqtt#16890 |
src/devices/awox.ts
Outdated
@@ -97,6 +104,27 @@ const definitions: Definition[] = [ | |||
'green', 'brightness_step_up', 'brightness_step_down', 'brightness_move_up', 'brightness_move_down', 'brightness_stop', | |||
'recall_1', 'color_temperature_step_up', 'color_temperature_step_down'])], | |||
}, | |||
{ | |||
zigbeeModel: ['ERCU_3groups_Zm'], | |||
fingerprint: [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can drop the fingerprint (identification goes via zigbeeModel)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did so. Hope I did it correctly :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
But for some reason Github did not update the PR accordingly... I am a bit lost, using GitLab normally :D
Thanks! |
Please review and test compile, I don't know much about TS so might have syntax errors :)