Skip to content

Remove timeout parameter for Telegram bot actions#41488

Merged
MartinHjelmare merged 1 commit into
home-assistant:nextfrom
hanwg:remove-timeout
Dec 9, 2025
Merged

Remove timeout parameter for Telegram bot actions#41488
MartinHjelmare merged 1 commit into
home-assistant:nextfrom
hanwg:remove-timeout

Conversation

@hanwg
Copy link
Copy Markdown
Contributor

@hanwg hanwg commented Oct 27, 2025

Proposed change

The docs for the timeout parameter mention that it is timeout for sending media. However, this timeout is actually a read timeout - the amount of time where the integration will wait while waiting for a response from the Telegram servers.

By default, there are no read/write timeouts so most users will not encounter timeout problems.
(Telegram server-side will still have timeouts so actions won't be waiting indefinitely)

The current timeout parameter is confusing and doesn't fulfill it's intended purpose.
This PR removes this parameter.

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

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 Hacktoberfest An PR on this issue (or the PR itself) is eligible towards Hacktoberfest! has-parent This PR has a parent PR in another repo next This PR goes into the next branch labels Oct 27, 2025
@netlify
Copy link
Copy Markdown

netlify Bot commented Oct 27, 2025

Deploy Preview for home-assistant-docs ready!

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

@hanwg hanwg marked this pull request as ready for review October 27, 2025 12:43
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, @hanwg 👍

@c0ffeeca7 c0ffeeca7 added the awaits-parent Awaits the merge of an parent PR label Oct 28, 2025
@home-assistant home-assistant Bot added the parent-merged The parent PR has been merged already label Dec 9, 2025
@MartinHjelmare MartinHjelmare merged commit ce70692 into home-assistant:next Dec 9, 2025
7 checks passed
@home-assistant home-assistant Bot removed parent-merged The parent PR has been merged already awaits-parent Awaits the merge of an parent PR labels Dec 9, 2025
@github-actions github-actions Bot locked and limited conversation to collaborators Dec 10, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Hacktoberfest An PR on this issue (or the PR itself) is eligible towards Hacktoberfest! has-parent This PR has a parent PR in another repo next This PR goes into the next branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants