Skip to content

Use secondary thermostat address for HVAC JSON#4

Open
devprbtt wants to merge 2 commits intomainfrom
codex/create-hvac-control-profile-for-json-commands-osnpbc
Open

Use secondary thermostat address for HVAC JSON#4
devprbtt wants to merge 2 commits intomainfrom
codex/create-hvac-control-profile-for-json-commands-osnpbc

Conversation

@devprbtt
Copy link
Copy Markdown
Owner

@devprbtt devprbtt commented Feb 2, 2026

Motivation

  • Recent removal of hard-coded zone suffixes broke Savant UI feedback for zone sliders and setpoints when thermostat IDs > 1, and the UI expects zone-1 bindings for controls to render properly.
  • The intent is to preserve UI/zone bindings while still allowing commands to target arbitrary device IDs.

Description

  • Restored zone-1 state center bindings (state_center_binding names with trailing _1) so the Savant UI slider and per-zone feedback work correctly.
  • Added a second address component and action argument ThermostatAddress2 to carry the actual device id, and left ThermostatAddress as the UI zone identifier (note: use 1 for the UI zone).
  • Updated all HVAC actions to send JSON commands using ThermostatAddress2 in the id field while preserving optimistic updates and HasReceivedCommand behavior.
  • Bumped profile revision to 1.3 and added notes documenting the two-address usage and the rationale.

Testing

  • No automated tests were executed for this profile-only change.
  • Changes were applied to the profile XML and syntax inspected locally; no runtime integration tests were run.

Codex Task

Updated HVAC control interfaces to include TCP protocol and adjusted response time. Added audio and video media elements.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant