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

Improve support for the SUTON STB3L-125-ZJ DIN rail RCBO (TZE204_wbha… #8011

Merged
merged 1 commit into from
Sep 23, 2024

Conversation

octa22
Copy link
Contributor

@octa22 octa22 commented Sep 21, 2024

This PR improves support for the SUTON STB3L-125-ZJ DIN rail RCBO (TZE204_wbhaespm)

@@ -5529,7 +5669,7 @@ const definitions: DefinitionWithExtend[] = [
vendor: 'SUTON',
description: 'Zigbee DIN RCBO energy meter',
fromZigbee: [tuya.fz.datapoints],
toZigbee: [tuya.tz.datapoints],
toZigbee: [tzLocal.TZE204_threshold, tuya.tz.datapoints],
Copy link
Owner

Choose a reason for hiding this comment

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

Please use just tuya.tz.datapoints instead of TZE204_threshold

Copy link
Contributor Author

Choose a reason for hiding this comment

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

OK, you mean to implement the logic in tuya.valueConvertor? Is there an example how to implement a setting logic for several exposed attributes behind a single dataPoint? Thanks

Copy link
Owner

Choose a reason for hiding this comment

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

Yes, here is an example to send data points directly from the converter (

await sendDataPointEnum(entity, 2, 1, 'dataRequest', 1); // manual
). With this you ca also send multiple.

@octa22 octa22 force-pushed the _TZE204_wbhaespm branch 3 times, most recently from 6998fe7 to ea216ea Compare September 23, 2024 11:42
…espm)

Signed-off-by: Ondrej Pecta <opecta@gmail.com>
@Koenkk Koenkk merged commit 93dbdfc into Koenkk:master Sep 23, 2024
2 checks passed
@Koenkk
Copy link
Owner

Koenkk commented Sep 23, 2024

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