Skip to content

Add FAQ for energy dashboard separate solar grid connection#45756

Merged
c0ffeeca7 merged 4 commits into
currentfrom
energy_separate_solar
Jun 8, 2026
Merged

Add FAQ for energy dashboard separate solar grid connection#45756
c0ffeeca7 merged 4 commits into
currentfrom
energy_separate_solar

Conversation

@abmantis

@abmantis abmantis commented Jun 4, 2026

Copy link
Copy Markdown
Member

Proposed change

Document this configuration since it deviates a bit from the usual ones.

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

  • Link to parent pull request in the codebase:
  • Link to parent pull request in the Brands repository:
  • This PR fixes or closes issue: fixes #

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 June 4, 2026 21:01
@home-assistant home-assistant Bot added the current This PR goes into the current branch label Jun 4, 2026
@netlify

netlify Bot commented Jun 4, 2026

Copy link
Copy Markdown

Deploy Preview for home-assistant-docs ready!

Name Link
🔨 Latest commit 5b9645a
🔍 Latest deploy log https://app.netlify.com/projects/home-assistant-docs/deploys/6a25d77295abb3000866e4f1
😎 Deploy Preview https://deploy-preview-45756--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 expands the Energy documentation FAQ with guidance for configuring the Energy dashboard when your solar production exports through a dedicated grid connection separate from your household consumption connection.

Changes:

  • Added a new FAQ section describing a two-grid-connection setup for solar export.
  • Documented the recommended Energy dashboard configuration fields to attribute flows as Solar → Grid.

Comment thread source/_docs/energy/faq.markdown Outdated
Comment thread source/_docs/energy/faq.markdown Outdated
Comment thread source/_docs/energy/faq.markdown Outdated
c0ffeeca7 and others added 2 commits June 5, 2026 07:18
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Comment thread source/_docs/energy/faq.markdown Outdated
@home-assistant home-assistant Bot marked this pull request as draft June 5, 2026 05:23
@home-assistant

home-assistant Bot commented Jun 5, 2026

Copy link
Copy Markdown

Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍

Learn more about our pull request process.

Co-authored-by: c0ffeeca7 <38767475+c0ffeeca7@users.noreply.github.com>
@abmantis

abmantis commented Jun 7, 2026

Copy link
Copy Markdown
Member Author

@c0ffeeca7 you removed the spaces around the em dash in 928606d. Is that intentional? It was already using em-dashes.

I think it looks like a hyphenated word without spaces:
image

@abmantis abmantis requested a review from c0ffeeca7 June 7, 2026 20:50
@c0ffeeca7

c0ffeeca7 commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

@c0ffeeca7 you removed the spaces around the em dash in 928606d. Is that intentional? It was already using em-dashes.

I think it looks like a hyphenated word without spaces: image

Yes, we use Merriam-Webster style (https://www.merriam-webster.com/grammar/em-dash-en-dash-how-to-use).
Microsoft Style Guide uses Merriam-Webster, and we follow MSG and Merriam-Webster.
Personally, I would use parentheses in this particular sentence, not an em dash.

Our developer docs already state that we use Merriam-Webster style. Added an additional entry here to make it more explicit: home-assistant/developers.home-assistant#3133

@c0ffeeca7 c0ffeeca7 marked this pull request as ready for review June 8, 2026 14:43

@c0ffeeca7 c0ffeeca7 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.

Thank you, @abmantis 👍

@c0ffeeca7 c0ffeeca7 merged commit 27cf6b9 into current Jun 8, 2026
9 checks passed
@c0ffeeca7 c0ffeeca7 deleted the energy_separate_solar branch June 8, 2026 14:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

current This PR goes into the current branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants