-
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
Update efekta.ts #8028
base: master
Are you sure you want to change the base?
Update efekta.ts #8028
Conversation
added support for two new devices EFEKTA PST DUO V1 LR and EFEKTA PST DUO V1
added support for two new devices EFEKTA PST DUO V1 LR and EFEKTA PST DUO V1
added support for two new devices EFEKTA PST DUO V1 LR and EFEKTA PST DUO V1
added support for two new devices EFEKTA PST DUO V1 LR and EFEKTA PST DUO V1
added support for two new devices EFEKTA PST DUO V1 LR and EFEKTA PST DUO V1
added support for two new devices EFEKTA PST DUO V1 LR and EFEKTA PST DUO V1
added support for two new devices EFEKTA PST DUO V1 LR and EFEKTA PST DUO V1
added support for two new devices EFEKTA PST DUO V1 LR and EFEKTA PST DUO V1
added support for two new devices EFEKTA PST DUO V1 LR and EFEKTA PST DUO V1
added support for two new devices EFEKTA PST DUO V1 LR and EFEKTA PST DUO V1
configure: async (device, coordinatorEndpoint, logger) => { | ||
const endpointOne = device.getEndpoint(1); | ||
const overrides0 = {min: 7200, max: 43200, change: 1}; | ||
await reporting.batteryAlarmState(endpointOne, overrides0); |
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.
I think this can also be added as a modernExtend?
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.
That would be great, it doesn't work through the modern right now. As soon as you implement this, I will change efekta.ts
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.
Could you check if you can add this? Add batteryAlarmStateReporting
to https://github.com/Koenkk/zigbee-herdsman-converters/blob/master/src/lib/modernExtend.ts#L285
added support for two new devices EFEKTA PST DUO V1 LR and EFEKTA PST DUO V1