Skip to content

Remove volvooncall #40504

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 2 commits into
base: next
Choose a base branch
from
Open

Remove volvooncall #40504

wants to merge 2 commits into from

Conversation

svrooij
Copy link

@svrooij svrooij commented Aug 18, 2025

Proposed change

  • Removing superseded integration volvooncall (replaced by volvo)

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: Remove volvooncall core#150725
  • 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 Copilot AI review requested due to automatic review settings August 18, 2025 21:27
@svrooij svrooij requested review from a team, jjlawren and Ernst79 as code owners August 18, 2025 21:27
@home-assistant home-assistant bot added current This PR goes into the current branch has-parent This PR has a parent PR in another repo 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 labels Aug 18, 2025
@home-assistant
Copy link

It seems that this PR is targeted against an incorrect branch since it has a parent PR on one of our codebases. Documentation that needs to be updated for an upcoming release should target the next branch. Please change the target branch of this PR to next and rebase if needed.

@netlify Netlify
Copy link

netlify bot commented Aug 18, 2025 •
edited
Loading

✅ Deploy Preview for home-assistant-docs ready!

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

@svrooij svrooij changed the base branch from current to next August 18, 2025 21:27
@home-assistant home-assistant bot removed needs-rebase The PR has been branched of the wrong base branch or targets an incorrect target branch current This PR goes into the current branch labels Aug 18, 2025
@home-assistant home-assistant bot added the next This PR goes into the next branch label Aug 18, 2025
Copilot
Copilot AI reviewed Aug 18, 2025
View reviewed changes
Copy link
Contributor

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

epenet
epenet requested changes Aug 19, 2025
View reviewed changes
Copy link
Contributor

@epenet epenet left a comment

Choose a reason for hiding this comment

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

Please make sure to update source/_redirects
See #39738 as an example

@home-assistant home-assistant bot marked this pull request as draft August 19, 2025 06:24
@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.

@epenet epenet changed the title Remove volvo 2 Remove volvooncall Aug 19, 2025
@svrooij
Copy link
Author

svrooij commented Aug 19, 2025

Please make sure to update source/_redirects
See #39738 as an example

This integration is replaced, would you say I make a redirect to the new integration? Or to the page this integration was removed?

@gjohansson-ST
Copy link
Member

This integration is replaced, would you say I make a redirect to the new integration? Or to the page this integration was removed?

That it's removed.

This was referenced Aug 20, 2025
Open
Open
@gjohansson-ST gjohansson-ST marked this pull request as ready for review August 20, 2025 16:22
@home-assistant home-assistant bot requested a review from epenet August 20, 2025 16:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

Copilot code review Copilot Copilot left review comments

@epenet epenet Awaiting requested review from epenet

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
has-parent This PR has a parent PR in another repo in-progress This PR/Issue is currently being worked on next This PR goes into the next branch
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@svrooij @gjohansson-ST @epenet