Skip to content

Conversation

@StefaniOSApps
Copy link

@StefaniOSApps StefaniOSApps commented Nov 30, 2025

Proposed change

Add documentation for the Energy Tracker integration.

Energy Tracker is an energy monitoring service with over 100,000 users that allows automated meter reading submissions from Home Assistant. This documentation covers setup, configuration, service usage, and troubleshooting.

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 November 30, 2025 11:19
@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 Nov 30, 2025
@netlify
Copy link

netlify bot commented Nov 30, 2025

Deploy Preview for home-assistant-docs ready!

Name Link
🔨 Latest commit 4b6c29b
🔍 Latest deploy log https://app.netlify.com/projects/home-assistant-docs/deploys/69305eb848bf0c0008b4711a
😎 Deploy Preview https://deploy-preview-42031--home-assistant-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Contributor

Copilot AI left a comment

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 adds comprehensive documentation for the new Energy Tracker integration, which enables Home Assistant users to automatically submit meter readings to their Energy Tracker service accounts. The documentation covers the complete user journey from initial setup through configuration, usage, and troubleshooting.

Key changes

  • Complete integration documentation following Home Assistant documentation standards
  • Prerequisites and setup instructions with clear steps for obtaining API tokens
  • Action documentation for sending meter readings with detailed parameter descriptions
  • Practical automation examples for daily and state-change-based meter reading submissions

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.

@StefaniOSApps
Copy link
Author

@copilot open a new pull request to apply changes based on the comments in this thread

@c0ffeeca7 c0ffeeca7 added the new-integration This PR adds documentation for a new Home Assistant integration label Dec 2, 2025
@home-assistant home-assistant bot marked this pull request as draft December 2, 2025 07:51
@home-assistant
Copy link

home-assistant bot commented Dec 2, 2025

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.

@StefaniOSApps StefaniOSApps marked this pull request as ready for review December 2, 2025 10:05
Copy link
Contributor

@c0ffeeca7 c0ffeeca7 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

found some more tiny tweaks

@home-assistant home-assistant bot marked this pull request as draft December 2, 2025 15:03
@StefaniOSApps StefaniOSApps marked this pull request as ready for review December 2, 2025 17:55
@StefaniOSApps
Copy link
Author

StefaniOSApps commented Dec 2, 2025

Hi @c0ffeeca7, thanks again for taking the time to read through the doc and giving such a detailed review.

If you have a moment, would you also be willing to take a look at the core PR? home-assistant/core#157571 The integration itself is quite straightforward (it simply sends sensor values to a backend).
The API logic is split out into the energy-tracker-api module (https://github.com/energy-tracker/api-client-python); most of the file changes come from translations, which I know can be off-putting in PR diffs.

Thanks!

@c0ffeeca7
Copy link
Contributor

Hi @c0ffeeca7, thanks again for taking the time to read through the doc and giving such a detailed review.

❤️

Sidenote: please don't tag reviewers.

If you have a moment, would you also be willing to take a look at the core PR? home-assistant/core#157571 The integration itself is quite straightforward (it simply sends sensor values to a backend). The API logic is split out into the energy-tracker-api module (https://github.com/energy-tracker/api-client-python); most of the file changes come from translations, which I know can be off-putting in PR diffs.

Thanks!

No, I can't help you there, I'm not a core reviewer. Please be patient. ❤️

c0ffeeca7
c0ffeeca7 previously approved these changes Dec 3, 2025
Copy link
Contributor

@c0ffeeca7 c0ffeeca7 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, @StefaniOSApps 👍

@c0ffeeca7 c0ffeeca7 added the awaits-parent Awaits the merge of an parent PR label Dec 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaits-parent Awaits the merge of an parent PR has-parent This PR has a parent PR in another repo new-integration This PR adds documentation for a new Home Assistant integration next This PR goes into the next branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants