Skip to content

docs(solac): Document manual inverter select - #46787

Closed
kdehairy wants to merge 3 commits into
home-assistant:nextfrom
kdehairy:feat/solax-integration-manual-inverter-select
Closed

docs(solac): Document manual inverter select#46787
kdehairy wants to merge 3 commits into
home-assistant:nextfrom
kdehairy:feat/solax-integration-manual-inverter-select

Conversation

@kdehairy

Copy link
Copy Markdown
Contributor

linked to this PR home-assistant/core#176275

Proposed change

In Solax integration, the current auto-discovery behavior often results in multiple "valid" inverters, and it just picks one, effectively, at random.

This patch allows the user to hand pick the model. If they did so, the auto-discovery is disabled and the hand picked model is chosen.

If left empty, the current auto-discovery behavior is picked up. making this backward compatible (UX wise).

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 review requested due to automatic review settings July 11, 2026 14:55
@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 Jul 11, 2026
@netlify

netlify Bot commented Jul 11, 2026

Copy link
Copy Markdown

Deploy Preview for home-assistant-docs ready!

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

This PR updates the Home Assistant website documentation for the SolaX Power integration to describe a new setup option that lets you manually select an inverter model, as introduced in the linked core PR (home-assistant/core#176275).

Changes:

  • Added a new documentation section describing manual inverter model selection during the config flow.
  • Explained when to use manual selection versus leaving the field empty to keep automatic detection.

Comment thread source/_integrations/solax.markdown
@kdehairy

Copy link
Copy Markdown
Contributor Author

I failed to find a "reopen" action on this PR. It is still needed.

The "Bot" closed it by mistake because it was linked to "another" PR that has been closed. My codebase PR linked here is still open.

Can you please help me reopen it?

@kdehairy

Copy link
Copy Markdown
Contributor Author

opened another one #46830 same content

@github-actions github-actions Bot locked and limited conversation to collaborators Jul 14, 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.

2 participants