Skip to content

fix: honor Telegram typing indicator setting - #49024

Closed
marcoleejr wants to merge 2 commits into
NousResearch:mainfrom
marcoleejr:fix/telegram-typing-indicator-setting
Closed

fix: honor Telegram typing indicator setting#49024
marcoleejr wants to merge 2 commits into
NousResearch:mainfrom
marcoleejr:fix/telegram-typing-indicator-setting

Conversation

@marcoleejr

Copy link
Copy Markdown

Summary

  • honor the existing typing_indicator display setting before starting background typing loops
  • gate direct Telegram typing calls and progress/follow-up typing pings on the same setting
  • add regression coverage for disabled Telegram typing indicators in both the base gateway flow and direct Telegram adapter calls

Testing

  • python -m pytest tests/gateway/test_base_topic_sessions.py tests/gateway/test_display_config.py tests/gateway/test_telegram_thread_fallback.py -q

@alt-glitch alt-glitch added type/feature New feature or request comp/gateway Gateway runner, session dispatch, delivery platform/telegram Telegram bot adapter P3 Low — cosmetic, nice to have labels Jun 19, 2026
@teknium1

Copy link
Copy Markdown
Contributor

Superseded by #55394 (merged to main, commit 05ac167). That PR adds a generic per-platform typing_indicator flag at the shared _keep_typing spawn site, covering Telegram and every other platform via typing_indicator: false. Thanks for the contribution — credited in the merged PR.

@teknium1 teknium1 closed this Jun 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/gateway Gateway runner, session dispatch, delivery P3 Low — cosmetic, nice to have platform/telegram Telegram bot adapter type/feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants