Skip to content

Add reconfigure section to NeoPool docs - #47635

Merged
silamon merged 3 commits into
home-assistant:nextfrom
svasek:neopool-reconfigure
Aug 27, 2026
Merged

Add reconfigure section to NeoPool docs#47635
silamon merged 3 commits into
home-assistant:nextfrom
svasek:neopool-reconfigure

Conversation

@svasek

@svasek svasek commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Proposed change

Document the reconfigure flow added to the NeoPool integration. Adds a Reconfigure section describing how to update an existing entry's connection settings (host, port, unit ID, Modbus framer) from the integration menu, and notes that a serial-number mismatch is rejected to prevent pointing the entry at a different controller.

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.

…atinum

- Document the reconfigure flow for updating connection settings
- Note the serial-mismatch guard against retargeting a different controller
- Raise ha_quality_scale from silver to platinum
Copilot AI lite review requested due to automatic review settings August 26, 2026 18:57
@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 26, 2026
@netlify

netlify Bot commented Aug 26, 2026

Copy link
Copy Markdown

Deploy Preview for home-assistant-docs ready!

Name Link
🔨 Latest commit b8768ea
🔍 Latest deploy log https://app.netlify.com/projects/home-assistant-docs/deploys/6a90077b07447f00089ac75a
😎 Deploy Preview https://deploy-preview-47635--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 NeoPool reconfiguration flow (added in Core) so readers can update connection settings from the UI, and updates the documented integration quality scale to reflect the integration’s current status.

Changes:

  • Updated ha_quality_scale from silver to platinum.
  • Added a Reconfigure section describing how to update host/port/unit ID/framer via the integration menu, including serial-number mismatch protection.
Suppressed comments (1)

source/_integrations/neopool.markdown:163

  • "the device reached at the new address" reads like a typo/grammar issue. It should be "the device at the new address".
The integration verifies the new settings against the controller before saving. If the device reached at the new address reports a different serial number than the one originally configured, the reconfiguration is rejected to prevent pointing the entry at a different controller.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread source/_integrations/neopool.markdown
@svasek
svasek marked this pull request as ready for review August 26, 2026 20:33
@svasek svasek changed the title Add reconfigure section to NeoPool docs and raise quality scale to platinum Add reconfigure section to NeoPool docs Aug 27, 2026
@home-assistant home-assistant Bot added the parent-merged The parent PR has been merged already label Aug 27, 2026
@silamon
silamon merged commit 1f0aa09 into home-assistant:next Aug 27, 2026
7 checks passed
@home-assistant home-assistant Bot removed the parent-merged The parent PR has been merged already label Aug 27, 2026
@svasek
svasek deleted the neopool-reconfigure branch August 27, 2026 15:44
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants