Swisscom documentation#45751
Merged
Merged
Conversation
✅ Deploy Preview for home-assistant-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Contributor
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Updates integration documentation and project dependencies to reflect current setup paths and tooling versions.
Changes:
- Refresh Swisscom Internet-Box integration docs to use config flow/UI setup and expand documentation sections.
- Clarify Indevolt integration limitations for specific models.
- Bump Ruby
jsondependency and updateactions/checkoutpin in CI workflow.
Reviewed changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| source/_integrations/swisscom.markdown | Modernizes Swisscom docs for config flow, adds prerequisites/functionality/troubleshooting sections. |
| source/_integrations/indevolt.markdown | Refines “Known limitations” bullets with model-specific notes. |
| Gemfile.lock | Updates locked json gem version. |
| .github/workflows/test.yml | Updates pinned SHA for actions/checkout. |
21 tasks
c0ffeeca7
reviewed
Jun 5, 2026
c0ffeeca7
requested changes
Jun 5, 2026
|
Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍 |
c0ffeeca7
reviewed
Jun 5, 2026
Co-authored-by: c0ffeeca7 <38767475+c0ffeeca7@users.noreply.github.com>
Author
|
Thanks for the suggestions! |
Co-authored-by: c0ffeeca7 <38767475+c0ffeeca7@users.noreply.github.com>
Co-authored-by: c0ffeeca7 <38767475+c0ffeeca7@users.noreply.github.com>
Co-authored-by: c0ffeeca7 <38767475+c0ffeeca7@users.noreply.github.com>
Contributor
|
Please undraft the PR once the changes have been implemented and the PR is ready for review again. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Proposed change
This updates the Swisscom Internet-Box documentation to match the revamp of the integration in the parent PR, which moves it from a YAML-only
device_trackerplatform to a UI-based config flow.The old page only described the legacy
configuration.yamlsetup, which no longer works reliably on recent Internet-Box firmware (the legacy platform queried the router unauthenticated). The integration now uses authenticated admin credentials, so the docs needed a rewrite to explain the new setup.What changed on the page:
configurationblock with the standard config flow include and aconfiguration_basicblock describing the Host, Username, and Password fields.ha_config_flow: trueand setha_integration_typetohubto match the new manifest.Type of change
currentbranch).currentbranch).nextbranch).nextbranch).Additional information
Checklist
currentbranch.nextbranch.