Skip to content

Document sensor platform for Hot Spring - #47542

Open
Moustachauve wants to merge 2 commits into
home-assistant:nextfrom
Moustachauve:hotspring-sensor
Open

Document sensor platform for Hot Spring#47542
Moustachauve wants to merge 2 commits into
home-assistant:nextfrom
Moustachauve:hotspring-sensor

Conversation

@Moustachauve

Copy link
Copy Markdown
Contributor

Proposed change

This adds the documentation for the new sensor platform for Hotspring integration.

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 to the existing documentation and used the current branch.
    • 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.

Copilot AI lite review requested due to automatic review settings August 21, 2026 04:16
@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 Aug 21, 2026
@netlify

netlify Bot commented Aug 21, 2026

Copy link
Copy Markdown

Deploy Preview for home-assistant-docs ready!

Name Link
🔨 Latest commit 917d65c
🔍 Latest deploy log https://app.netlify.com/projects/home-assistant-docs/deploys/6a87d30ffb4ed10008189e8a
😎 Deploy Preview https://deploy-preview-47542--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.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Documents the new Hot Spring sensor platform in Home Assistant.

Changes:

  • Adds sensor metadata to the integration page.
  • Documents temperature, salt-system, and firmware sensors.
  • Notes availability and default-disabled entities.
Suppressed comments (3)

source/_integrations/hotspring.markdown:75

  • Core only creates this entity when spa.versions.control_box is non-empty; its tests also cover the no-version case. As written, this says the entity is merely disabled, but on devices without a reported version it is absent. Please document that availability condition, as you do for the FreshWater sensor.
- **Control box version**
  - **Description**: Firmware version of the spa control box.
  - **Remarks**: Disabled by default.

source/_integrations/hotspring.markdown:81

  • Core only creates this entity when spa.versions.wifi_dongle is non-empty; its tests also cover the no-version case. As written, this says the entity is merely disabled, but on devices without a reported version it is absent. Please document that availability condition.
- **Wi-Fi dongle version**
  - **Description**: Firmware version of the Wi-Fi dongle.
  - **Remarks**: Disabled by default.

source/_integrations/hotspring.markdown:66

  • This availability note does not match the parent implementation's behavior: the three cartridge sensors are created only when cartridge_installed is true during platform setup, and that predicate is not re-evaluated on coordinator updates. Installing or removing a cartridge later therefore does not add or remove these entities (and removed cartridges can leave stale values). Please fix the parent implementation to track availability dynamically, or document that reloading the integration is required.
- **Salt 10-day check timer**
  - **Description**: Number of days remaining until the next 10-day salt water test reminder.
  - **Availability**: Available when a FreshWater Salt System cartridge is installed.

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

- **Description**: Allows setting the target water temperature for the spa.
- **Range**: 80 °F to 104 °F

### Sensor

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done.

@home-assistant home-assistant Bot added the parent-merged The parent PR has been merged already label Aug 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

has-parent This PR has a parent PR in another repo next This PR goes into the next branch parent-merged The parent PR has been merged already

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants