docs(solac): Document manual inverter select - #46786
Closed
kdehairy wants to merge 1 commit into
Closed
Conversation
home-assistant
Bot
added
current
✅ Deploy Preview for home-assistant-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
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 |
21 tasks
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the SolaX Power integration documentation to explain the newly added setup option for manually selecting an inverter model (to avoid unreliable auto-detection outcomes), as introduced in the linked Core PR.
Changes:
- Added a new documentation section describing manual inverter model selection during setup.
Comment on lines
+23
to
+25
| ### Manual inverter model selection | ||
|
|
||
| By default, this integration automatically detects your inverter model by probing every supported model in parallel. If auto-detection is unreliable for your device, you can bypass it by manually selecting your inverter model from the dropdown during setup. Leave this field blank to keep using automatic detection. |
linked to this PR home-assistant/core#176275
kdehairy
force-pushed
the
feat/solax-integration-manual-inverter-select
branch
from
July 11, 2026 14:52
ed235a5 to
d483c14
Compare
kdehairy
requested review from
a team,
CoMPaTech,
Danielhiversen,
Quentame,
Shulyaka,
StevenLooman,
arturpragacz,
bdraco,
cgtobi,
ctalkington,
dgomes,
emontnemery,
erwindouna,
fabaff,
mib1185,
raman325 and
zweckj
as code owners
July 11, 2026 14:52
kdehairy
requested review from
Swamp-Ig,
allenporter,
cgarwood,
chemelli74,
farmio,
jbouwh,
jesserockz,
kbx81 and
ochlocracy
as code owners
July 11, 2026 14:52
home-assistant
Bot
removed
current
Contributor
Author
|
Closing this one. mistakenly I based it on current, while I should've based it on next |
home-assistant
Bot
removed
the
in-progress
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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
currentbranch).currentbranch).nextbranch).nextbranch).Additional information
Checklist
currentbranch.nextbranch.