Skip to content

Document the Silla Prism charging mode select - #47928

Open
ebaschiera wants to merge 2 commits into
home-assistant:nextfrom
ebaschiera:silla-prism-docs-select
Open

Document the Silla Prism charging mode select#47928
ebaschiera wants to merge 2 commits into
home-assistant:nextfrom
ebaschiera:silla-prism-docs-select

Conversation

@ebaschiera

@ebaschiera ebaschiera commented Sep 6, 2026

Copy link
Copy Markdown
Contributor

Proposed change

Documents the charging mode select added to the Silla Prism integration, the
first of the follow-ups left out of the initial PR when it was reduced to a
single platform.

Adds the select platform and category, a Selects section describing the
three selectable modes, and the note that the Prism's own load-balancing pause
cannot be selected back, so the entity reads as unknown while it lasts. The
known limitation about the integration being read-only is narrowed to the
charging current, which is still not exposed.

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 September 6, 2026 06:36
@home-assistant home-assistant Bot added in-progress This PR/Issue is currently being worked on needs-rebase The PR has been branched of the wrong base branch or targets an incorrect target branch next This PR goes into the next branch labels Sep 6, 2026
@home-assistant

home-assistant Bot commented Sep 6, 2026

Copy link
Copy Markdown

It seems that this PR is targeted against an incorrect branch. Documentation updates which apply to our current stable release should target the current branch. Please change the target branch of this PR to current and rebase if needed. If this is documentation for a new feature, please add a link to that PR in your description.

@netlify

netlify Bot commented Sep 6, 2026

Copy link
Copy Markdown

Deploy Preview for home-assistant-docs ready!

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

@home-assistant home-assistant Bot added has-parent This PR has a parent PR in another repo and removed needs-rebase The PR has been branched of the wrong base branch or targets an incorrect target branch labels Sep 6, 2026

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.

🟡 Changes recommended

The new Charging mode description can be misread as implying Home Assistant sets the charging current and would benefit from clearer wording.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

This PR updates the Silla Prism integration documentation to reflect the newly added Charging mode select entity, aligning the docs with the integration’s expanded (sensor + select) functionality.

Changes:

  • Add Select to the integration metadata (ha_category, ha_platforms).
  • Add a Selects section documenting the Charging mode options and the load-balancing “unknown” behavior.
  • Update Known limitations to remove the now-implemented charging mode control and narrow the remaining limitation to charging current.
File summaries
File Description
source/_integrations/silla_prism.markdown Adds select metadata and documents the new Charging mode select while updating limitations accordingly.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 1
  • Review effort level: Lite

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


### Selects

- **Charging mode**: The operating mode of the port. **Solar** charges only with the surplus available, **Normal** charges at the set current, and **Paused** suspends charging without disconnecting the vehicle. The Prism also pauses charging on its own when load balancing runs out of available power; that condition cannot be selected back, so the charging mode reads as unknown while it lasts and the status sensor reports the port as paused.

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.

fixed

Split the modes into their own list, say that the charging current is set
on the wallbox, and drop the "selected back" wording for the mode the
Prism enters on its own.
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 in-progress This PR/Issue is currently being worked on next This PR goes into the next branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants