Skip to content

Add known limitions to devolo Home Control#41108

Merged
c0ffeeca7 merged 1 commit intohome-assistant:currentfrom
Shutgun:hc_known_limitations
Dec 19, 2025
Merged

Add known limitions to devolo Home Control#41108
c0ffeeca7 merged 1 commit intohome-assistant:currentfrom
Shutgun:hc_known_limitations

Conversation

@Shutgun
Copy link
Copy Markdown
Contributor

@Shutgun Shutgun commented Oct 1, 2025

Proposed change

Add a chapter about known limitations to devolo Home Control to fulfill the docs-known-limitations quality scale requirement.

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.

@home-assistant home-assistant Bot added current This PR goes into the current branch Hacktoberfest An PR on this issue (or the PR itself) is eligible towards Hacktoberfest! labels Oct 1, 2025
@netlify
Copy link
Copy Markdown

netlify Bot commented Oct 1, 2025

Deploy Preview for home-assistant-docs ready!

Name Link
🔨 Latest commit da7ba55
🔍 Latest deploy log https://app.netlify.com/projects/home-assistant-docs/deploys/68dd40803a91270008234bae
😎 Deploy Preview https://deploy-preview-41108--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.

frenck
frenck previously requested changes Oct 1, 2025
Comment on lines +118 to +119
Device addition and removal are not supported by this integration as the local API rejects these operations.
To add or remove devices, please use the devolo Home Control app or the web interface.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I don't think this is an limitation, more of an instruction on how to remove devices?

../Frenck

Copy link
Copy Markdown
Contributor Author

@Shutgun Shutgun Oct 1, 2025

Choose a reason for hiding this comment

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

My main motivation was to prevent some to think one can replace the devolo world completely with Home Assistant, especially under the aspect, that devolo will discontinue the app and the web interface end of this year.

Would you just remove the last sentence like in "the user probably knows what to do" or would you skip the complete section like in "Home Assistant users typically know that hub integrations need other ways to remove devices"? I don't use a hub integration in my productive setup so I don't have other references.

@home-assistant home-assistant Bot marked this pull request as draft October 1, 2025 17:49
@home-assistant
Copy link
Copy Markdown

home-assistant Bot commented Oct 1, 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.

@github-actions
Copy link
Copy Markdown

There hasn't been any activity on this pull request recently. This pull request has been automatically marked as stale because of that and will be closed if no further activity occurs within 7 days.
Thank you for your contributions.

@github-actions github-actions Bot added the stale The PR had no activity for quite some time, and is marked as Stale label Nov 30, 2025
@Shutgun
Copy link
Copy Markdown
Contributor Author

Shutgun commented Dec 2, 2025

It's waiting for feedback ...

@github-actions github-actions Bot removed the stale The PR had no activity for quite some time, and is marked as Stale label Dec 2, 2025
@silamon silamon requested a review from frenck December 7, 2025 08:48
@silamon
Copy link
Copy Markdown
Contributor

silamon commented Dec 7, 2025

I'm going to put Frenck back on the review list, it's mostly gone forgotten since you didn't mark it ready for review.
Thinking of it, if you can't remove devices from Home Assistant, doesn't it need to be on the removal instructions that you need to remove it from the web app or devolo app first?

@silamon silamon marked this pull request as ready for review December 7, 2025 08:48
Copy link
Copy Markdown
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, @Shutgun 👍

@c0ffeeca7 c0ffeeca7 dismissed frenck’s stale review December 19, 2025 10:05

to me, it makes sense to add this under limitations, as for other integrations I can add or remove devices directly in Home Assistant.

@c0ffeeca7 c0ffeeca7 merged commit 1d1c900 into home-assistant:current Dec 19, 2025
7 checks passed
@Shutgun Shutgun deleted the hc_known_limitations branch December 19, 2025 11:28
@github-actions github-actions Bot locked and limited conversation to collaborators Dec 20, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

current This PR goes into the current branch Hacktoberfest An PR on this issue (or the PR itself) is eligible towards Hacktoberfest!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants