-
Notifications
You must be signed in to change notification settings - Fork 5.7k
API 9.1 Checklist*
classes
#4848
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.
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
base: api-9.1
Are you sure you want to change the base?
Conversation
# Conflicts: # changes/unreleased/4847.8ujbbBbaZ2VTEdRLeqirSZ.toml
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Adds support for Bot API 9.1 “Checklist” types across the library, including models, filters, docs, and tests.
- Introduce new TelegramObject classes: Checklist, ChecklistTask, ChecklistTasksAdded, ChecklistTasksDone
- Expose checklist fields in
Message
andExternalReplyInfo
, update filters and changelog - Add comprehensive tests and Sphinx docs for the new classes
Reviewed Changes
Copilot reviewed 14 out of 14 changed files in this pull request and generated 2 comments.
Show a summary per file
File | Description |
---|---|
tests/test_reply.py | Added checklist to ExternalReplyInfo tests |
tests/test_message.py | Included checklist and tasks fixtures in message tests |
tests/test_checklists.py | New tests for Checklist, ChecklistTask, and service messages |
tests/ext/test_filters.py | Tests for filters.CHECKLIST and task‐based status filters |
src/telegram/ext/filters.py | Added CHECKLIST and status filters for checklist events |
src/telegram/_reply.py | Exposed checklist in ExternalReplyInfo |
src/telegram/_message.py | Added checklist* fields to Message |
src/telegram/_checklists.py | Implemented Checklist models |
src/telegram/init.py | Public exports for new classes |
docs/source/telegram.checklist*.rst | Documentation for Checklist and related classes |
docs/source/telegram.at-tree.rst | Added checklist types to the type tree |
changes/unreleased/4847*.toml | Updated changelog for checklist features |
Comments suppressed due to low confidence (1)
docs/source/telegram.at-tree.rst:35
- [nitpick] Add a dedicated documentation file for
ChecklistTask
(e.g.,telegram.checklisttask.rst
) so its members are fully documented alongside the other checklist types.
telegram.checklisttask
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes look good 🎊! Only a few docs fixes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just a single comment :)
No description provided.