Document Rabbit Air air quality sensor#45749
Conversation
|
It seems that this PR is targeted against an incorrect branch since it has a parent PR on one of our codebases. Documentation that needs to be updated for an upcoming release should target the |
✅ Deploy Preview for home-assistant-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Adds sensor support to the Rabbit Air integration documentation so the UI metadata and feature description reflect newly available air quality sensors.
Changes:
- Added
Sensortoha_category. - Added
sensortoha_platforms. - Updated the integration description to mention air quality sensor reporting.
| - A3 | ||
|
|
||
| The fan platform of this integration allows you to turn the unit on/off, select the preset mode, or set the speed manually. | ||
| The fan platform of this integration allows you to turn the unit on/off, select the preset mode, or set the speed manually. The sensor platform provides the current air quality reported by the device. |
|
Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍 |
Document the Rabbit Air air quality sensor added in home-assistant/core#172993. - [ ] Spelling, grammar or other readability improvements (`current` branch). - [x] Adjusted missing or incorrect information in the current documentation (`current` branch). - [ ] Added documentation for a new integration I'm adding to Home Assistant (`next` branch). - [ ] Added documentation for a new feature I'm adding to Home Assistant (`next` branch).
|
Apologies, should be sorted now. |
There was a problem hiding this comment.
Now that there are multiple platforms, maybe add a supported functionality section and add a Sensors section there?
You can find an example here:
https://github.com/home-assistant/home-assistant.io/blob/current/source/_integrations/_integration_docs_template.markdown?plain=1#L91
And the general page structure:
https://developers.home-assistant.io/docs/documenting/integration-docs-examples#basic-structure-of-an-integration-page
|
Sure will do - I am planning to expand the integration further next month so will make a good starting point. |
|
I added a Supported functionality section following the integration docs template, with separate Fans and Sensors subsections. The new Sensors section documents the Air quality sensor and its possible states: I also moved the supported model list into a Supported devices section to better match the standard integration page structure. |
Proposed change
Document the Rabbit Air air quality sensor added in home-assistant/core#172993.
Type of change
currentbranch).currentbranch).nextbranch).nextbranch).Additional information
Checklist
currentbranch.nextbranch.