Skip to content

Remove SALUS/Computime OTA from ZHA integration support list#42635

Merged
c0ffeeca7 merged 1 commit intohome-assistant:currentfrom
Hedda:patch-496866
Dec 18, 2025
Merged

Remove SALUS/Computime OTA from ZHA integration support list#42635
c0ffeeca7 merged 1 commit intohome-assistant:currentfrom
Hedda:patch-496866

Conversation

@Hedda
Copy link
Copy Markdown
Contributor

@Hedda Hedda commented Dec 18, 2025

Proposed change

Remove mentioning of SALUS/Computime OTA provider support from ZHA integration list of built-in Zigbee OTA providers. For reference; Support for SALUS/Computime OTA provider was "temporarly" removed in the upstream zigpy library that ZHA depends on for this more than a year ago due to old URL is dead but has not yet been updated with a new URL:

"This completely removes the Salus OTA provider from zigpy, as it no longer works due to Salus replacing their app and taking the links offline. Only the config options are kept, so existing configuration doesn't error out for now. In the future, we might be able to partially revert this if we have the new OTA link (assuming the structure is mostly the same)."

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.

Remove mentioning of SALUS/Computime OTA provider support from ZHA integration list of built-in Zigbee OTA providers.

For reference; Support for SALUS/Computime OTA provider was "temporarly" removed in the upstream zigpy a year ago due to old URL is dead but has not yet been updated with a new UR::

* zigpy/zigpy#1522

"***This completely removes the Salus OTA provider from zigpy, as it no longer works due to Salus replacing their app and taking the links offline. Only the config options are kept, so existing configuration doesn't error out for now. In the future, we might be able to partially revert this if we have the new OTA link (assuming the structure is mostly the same).***"
@home-assistant home-assistant Bot added current This PR goes into the current branch 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 Dec 18, 2025
@home-assistant
Copy link
Copy Markdown

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
Copy link
Copy Markdown

netlify Bot commented Dec 18, 2025

Deploy Preview for home-assistant-docs ready!

Name Link
🔨 Latest commit 5f8fa22
🔍 Latest deploy log https://app.netlify.com/projects/home-assistant-docs/deploys/6943c043786d2e0008bf7fa9
😎 Deploy Preview https://deploy-preview-42635--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
Copy Markdown
Member

@TheJulianJES TheJulianJES left a comment

Choose a reason for hiding this comment

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

Thanks!

@c0ffeeca7 c0ffeeca7 removed 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 Dec 18, 2025
@c0ffeeca7 c0ffeeca7 merged commit f4d64c9 into home-assistant:current Dec 18, 2025
7 checks passed
lukkigi pushed a commit to lukkigi/home-assistant.io that referenced this pull request Dec 18, 2025
@github-actions github-actions Bot locked and limited conversation to collaborators Dec 19, 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 second-opinion-wanted

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants