Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add device model #34929

Closed
wants to merge 48 commits into from
Closed

Add device model #34929

wants to merge 48 commits into from

Conversation

ashionky
Copy link
Contributor

@ashionky ashionky commented Sep 26, 2024

Proposed change

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.

Summary by CodeRabbit

  • New Features
    • Added support for the Refoss Smart Energy Monitor, EM16 in the Home Assistant integration documentation.
  • Documentation
    • Improved clarity of the supported device models table with a more consistent header layout.

@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 26, 2024
@home-assistant
Copy link

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.

@home-assistant home-assistant bot marked this pull request as draft September 30, 2024 13:37
@home-assistant
Copy link

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.

dxmnkd316 and others added 20 commits October 5, 2024 12:18
…e-assistant#35105)

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…35104)

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…35106)

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Tobias Klug <klug@werdenktwas.de>
@ashionky
Copy link
Contributor Author

ashionky commented Oct 8, 2024

There are unrelated commits in this PR:

CleanShot 2024-09-30 at 15 37 12@2x

Please git rebase your changes onto our latest dev branch in order to clean that up.

../Frenck

Ok, this is the branch I used before.
I will resubmit a PR; #35129 .
I will close it for now .
Thanks!

@ashionky ashionky closed this Oct 8, 2024
@home-assistant home-assistant bot removed the awaits-parent Awaits the merge of an parent PR label Oct 8, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Oct 9, 2024
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 a other repo next This PR goes into the next branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.