Add Home Assistant MQTT discovery for system trouble sensors#603
Open
cpavot wants to merge 1 commit into
Open
Conversation
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hello,
Note sure why troubles are currently not propagated from alarm panel to Home Assistant. I find it useful to notify me when something goes wrong (typically AC power failure troube). So let me know if I missed something (maybe something specific to my setup ?).
And thanks for the great job, that's awesome to have such a mature implementation !
Summary
This PR adds Home Assistant MQTT discovery for panel trouble states exposed by PAI.
PAI already publishes trouble states under:
paradox/states/system/troubles/...
but Home Assistant discovery was only generated from
status["system"]. On MG/SP panels, troubles are present in the status update payload as a top-levelstatus["troubles"]branch, so Home Assistant did not create the corresponding binary sensors.This PR normalizes the discovery input by adding the top-level
troublesbranch to the system discovery flow, producing entities such as:binary_sensor._system_troubles_ac_failure_trouble
binary_sensor._system_troubles_low_battery_trouble
binary_sensor._system_troubles_telephone_line
Why
This makes system trouble states directly available in Home Assistant without relying on raw event parsing or custom automations.
Notes
The existing
SystemBinarySensorimplementation and MQTT topic convention are reused. No new MQTT state topic structure is introduced.