Skip to content

Add webhook requirements section to Withings documentation#42536

Merged
joostlek merged 1 commit into
home-assistant:currentfrom
ptarjan:claude/withings-webhook-docs-013rJpb23oZwEQpNR4RFAD6F
Dec 16, 2025
Merged

Add webhook requirements section to Withings documentation#42536
joostlek merged 1 commit into
home-assistant:currentfrom
ptarjan:claude/withings-webhook-docs-013rJpb23oZwEQpNR4RFAD6F

Conversation

@ptarjan
Copy link
Copy Markdown
Contributor

@ptarjan ptarjan commented Dec 12, 2025

Document that webhooks require HTTPS specifically on port 443, not just any HTTPS port. This addresses a common configuration issue where users have HTTPS but on a non-standard port (e.g., 8443).

Also documents that sleep sensors require working webhooks and have no polling fallback, helping users understand why these sensors may be missing.

Ref: home-assistant/core#158748

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

  • 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.

Document that webhooks require HTTPS specifically on port 443, not just
any HTTPS port. This addresses a common configuration issue where users
have HTTPS but on a non-standard port (e.g., 8443).

Also documents that sleep sensors require working webhooks and have no
polling fallback, helping users understand why these sensors may be
missing.

Ref: home-assistant/core#158748
@ptarjan ptarjan requested a review from joostlek as a code owner December 12, 2025 16:06
@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 Dec 12, 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 12, 2025

Deploy Preview for home-assistant-docs ready!

Name Link
🔨 Latest commit 576f847
🔍 Latest deploy log https://app.netlify.com/projects/home-assistant-docs/deploys/693c3d9a2116840008757bb2
😎 Deploy Preview https://deploy-preview-42536--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.

@joostlek joostlek merged commit 905cc52 into home-assistant:current Dec 16, 2025
7 checks passed
@home-assistant home-assistant Bot 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 16, 2025
jstirnaman pushed a commit to jstirnaman/home-assistant.io that referenced this pull request Dec 16, 2025
@github-actions github-actions Bot locked and limited conversation to collaborators Dec 17, 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 has-parent This PR has a parent PR in another repo second-opinion-wanted

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants