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

[Enhancement] Report zone specific system trouble to the zone instead of the panel #64

Open
jimboca opened this issue Sep 19, 2022 · 2 comments
Assignees
Labels
enhancement New feature or request

Comments

@jimboca
Copy link
Contributor

jimboca commented Sep 19, 2022

Is your feature request related to a problem? Please describe.
Report zone specific system trouble to the zone instead of the panel?

Describe the solution you'd like
Currently this sends a zone specific system trouble to the panel, which the zone number:
'system_trouble_status': 'AC Fail, Box Tamper zone 8, Fail To Communicate, EEProm Memory Error'
I was going to add support for reporting this to a zone in my ISY integration, but was wondering if we could change elkm1_lib to report these to the zone instead of users having to parse the 'zone ' string?

Describe alternatives you've considered
Parse the string to get the zone number out of the 'zone '.

Additional context
Let me know what you think, if you would prefer to keep it as is then we can close and I will handle it on my end.

@jimboca jimboca added the enhancement New feature or request label Sep 19, 2022
@gwww
Copy link
Owner

gwww commented Sep 19, 2022

I'm open to considering this. What are your thoughts on how to handle when a zone has multiple trouble conditions?

@jimboca
Copy link
Contributor Author

jimboca commented Sep 24, 2022

I'm not sure the best way to handle it, The way it's handled on the panel makes sense where the user is sent a comma separated list, but it's really not ideal. It would be nice to just have properties available for each, but I understand why all are sent in at the same time, so it works. I was thinking it should be done the same way for zones unless you have better ideas?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants