Skip to content

Add sensor documentation to Duco integration#45039

Merged
c0ffeeca7 merged 3 commits into
home-assistant:nextfrom
ronaldvdmeer:add-duco-ventilation-sensors-docs
Apr 30, 2026
Merged

Add sensor documentation to Duco integration#45039
c0ffeeca7 merged 3 commits into
home-assistant:nextfrom
ronaldvdmeer:add-duco-ventilation-sensors-docs

Conversation

@ronaldvdmeer
Copy link
Copy Markdown
Contributor

@ronaldvdmeer ronaldvdmeer commented Apr 27, 2026

Proposed change

Extends the Duco integration documentation with new sensor entities:

Temperature sensor (all node types)

  • Available for UCCO2, BSRH, UCRH, and the BOX node (disabled by default for BOX because it measures housing temperature, not room temperature).

Target flow level sensor (BOX)

  • Shows the actual airflow target as a percentage (0–100 %).
  • Explains why this differs from the abstract 33/66/100 % levels shown in the fan entity.

Mode time remaining sensor (BOX)

  • Shows the seconds remaining in the current timed ventilation mode; 0 when no timer is active.

Also adds a note to the fan section clarifying the abstract speed levels and pointing users to the Target flow level sensor for the real firmware value.

Type of change

  • Spelling, grammar or other readability improvements (current branch).
  • 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).
  • Removed stale or deprecated documentation.

Additional information

Checklist

  • This PR uses the correct branch, based on one of the following:
    • I made a change that is related to an upcoming version of Home Assistant and used the next branch.
  • The documentation follows the Home Assistant documentation standards.

@home-assistant home-assistant Bot added has-parent This PR has a parent PR in another repo next This PR goes into the next branch labels Apr 27, 2026
@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 27, 2026

Deploy Preview for home-assistant-docs ready!

Name Link
🔨 Latest commit f4fa92a
🔍 Latest deploy log https://app.netlify.com/projects/home-assistant-docs/deploys/69f3370773c7170008ef2c3f
😎 Deploy Preview https://deploy-preview-45039--home-assistant-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@ronaldvdmeer ronaldvdmeer force-pushed the add-duco-ventilation-sensors-docs branch from d10033d to 5eeebc0 Compare April 27, 2026 14:59
@ronaldvdmeer ronaldvdmeer marked this pull request as ready for review April 27, 2026 15:03
Copilot AI review requested due to automatic review settings April 27, 2026 15:03
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the Duco integration documentation to cover newly added sensor entities (target airflow, timer remaining, and expanded temperature details), and clarifies how fan percentage levels relate to actual firmware airflow targets.

Changes:

  • Expands the BOX module description to include the new sensor entities.
  • Adds a note explaining that the fan’s 33/66/100% are abstract UI levels and points to the new “Target flow level” sensor.
  • Documents the new “Target flow level” and “Mode time remaining” sensors under the Sensors section.

Comment thread source/_integrations/duco.markdown Outdated
Comment thread source/_integrations/duco.markdown Outdated
Comment thread source/_integrations/duco.markdown Outdated
Comment thread source/_integrations/duco.markdown Outdated
@home-assistant home-assistant Bot marked this pull request as draft April 30, 2026 09:43
@home-assistant
Copy link
Copy Markdown

Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍

Learn more about our pull request process.

@home-assistant home-assistant Bot added the parent-merged The parent PR has been merged already label Apr 30, 2026
The sensor was renamed in the core PR from a countdown sensor
(seconds remaining) to a timestamp sensor (when the mode ends).
Update all three occurrences in the docs and revise the description
to reflect the new behavior.

Addresses review feedback from c0ffeeca7.
@ronaldvdmeer
Copy link
Copy Markdown
Contributor Author

ronaldvdmeer commented Apr 30, 2026

Thank you @c0ffeeca7 for the review. All changed.

Caution

The core PR is for the 2026.6 release. Not for the upcoming 2026.5 release.

Copy link
Copy Markdown
Contributor

@c0ffeeca7 c0ffeeca7 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, @ronaldvdmeer 👍

@c0ffeeca7 c0ffeeca7 added awaits-parent Awaits the merge of an parent PR and removed parent-merged The parent PR has been merged already labels Apr 30, 2026
@c0ffeeca7 c0ffeeca7 marked this pull request as ready for review April 30, 2026 11:16
@home-assistant home-assistant Bot requested a review from c0ffeeca7 April 30, 2026 11:16
@c0ffeeca7 c0ffeeca7 merged commit 369a4f9 into home-assistant:next Apr 30, 2026
7 checks passed
@home-assistant home-assistant Bot removed the awaits-parent Awaits the merge of an parent PR label Apr 30, 2026
@ronaldvdmeer ronaldvdmeer deleted the add-duco-ventilation-sensors-docs branch April 30, 2026 11:25
@github-actions github-actions Bot locked and limited conversation to collaborators May 1, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

has-parent This PR has a parent PR in another repo next This PR goes into the next branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants