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

Device/tuya bac 002 #8694

Merged
merged 3 commits into from
Jan 29, 2025
Merged

Conversation

robvanoostenrijk
Copy link
Contributor

Refactored BAC-002-AZLB / BAC-003:

  • Added device specific setting to indicate if 2-pipe (cooling / fan) or 4-pipe (cooling, heating & fan) mode is configured on the device, adjusting available system_mode options accordingly
  • Updated onEvent to onEventSetLocalTime correcting the device day / time as reported in Making BAC-002-ALZB Thermostats Work Flawlessly with Zigbee2MQTT zigbee2mqtt#25718
  • Added meta publishDuplicateTransaction: true, as device originated messages are always seqId 1. (This fixes broken device updates (not reflecting) in zigbee2mqtt somewhere recently, possibly version 2.x upgrade?)
  • Removed state attribute entirely, as Home Assistant and others use system_mode for climate controls
  • Refactored schedule format and parsing logic somewhere more maintainable
  • Added max_temperature datapoint from device service menu
  • Removed current_cooling_setpoint as it confused downstream systems, though intended to fix Matter integration
  • Tidied up DP#2 to use a single meta configuration entry

@Koenkk Koenkk merged commit 9a94890 into Koenkk:master Jan 29, 2025
2 checks passed
@Koenkk
Copy link
Owner

Koenkk commented Jan 29, 2025

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