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

Add compatibility for 3-Groups RCU from EGLO #5853

Merged
merged 6 commits into from
Jun 7, 2023

Conversation

fsedarkalex
Copy link
Contributor

Please review and test compile, I don't know much about TS so might have syntax errors :)

@fsedarkalex
Copy link
Contributor Author

fsedarkalex commented Jun 7, 2023

@Koenkk this is the requested PR for Koenkk/zigbee2mqtt#16890

@@ -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: [
Copy link
Owner

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)

Copy link
Contributor Author

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 :)

Copy link
Contributor Author

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

@Koenkk Koenkk merged commit 1bf3f61 into Koenkk:master Jun 7, 2023
@Koenkk
Copy link
Owner

Koenkk commented Jun 7, 2023

Thanks!

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.

2 participants