-
-
Notifications
You must be signed in to change notification settings - Fork 31.6k
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
Add Total Volatile Organic Compounds (tVOC) matter discovery schema #116963
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍 |
Hey there @home-assistant/matter, mind taking a look at this pull request as it has been labeled with an integration ( Code owner commandsCode owners of
|
Thanks for the contribution! The discovery schema looks fine, thanks! |
Adds a little bit of missing configuration to propagate tVOC readings from a Matter device to Home Assistant. Tested with SenseCAP indicator with indicator-matter firmware. NOTE: Device needs to be deleted and re-added for the sensor to appear.
A recently added fixture also contained the tVOC measurement so I have just added a test. |
Proposed change
Adds a little bit of missing configuration to propagate tVOC readings from a Matter device to Home Assistant. Without this patch tVOC readings are ignored. Tested with SenseCAP indicator with indicator-matter firmware.
NOTE: Device needs to be deleted and re-added for the sensor to appear.
If this pull request is not of adequate quality, please consider this to be a feature request to add tVOC sensor to matter integration.
Type of change
Additional information
This change would also be needed for #105567
Checklist
ruff format homeassistant tests
)To help with the load of incoming pull requests: