-
Notifications
You must be signed in to change notification settings - Fork 5.8k
API 9.1 InputChecklist[Task]
classes and bot.send/edit_checklist
#4857
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
Conversation
Note: | ||
You can only edit messages that the bot sent itself (i.e. of the ``bot.send_*`` family | ||
of methods) or channel posts, if the bot is an admin in that channel. However, this | ||
behaviour is undocumented and might be changed by Telegram. |
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.
Was hesitant about this note section, whether it's relevant here, should be removed entirely, or made to say something like:
Note: | |
You can only edit messages that the bot sent itself (i.e. of the ``bot.send_*`` family | |
of methods) or channel posts, if the bot is an admin in that channel. However, this | |
behaviour is undocumented and might be changed by Telegram. | |
Note: | |
You can only edit messages that the bot sent itself (i.e. of the ``bot.send_*`` family | |
of methods). However, this | |
behaviour is undocumented and might be changed by Telegram. |
It exists in most, if not all, other Message.edit_
shortcuts
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.
If the text is mostly the same everywhere, I'd vote to keep it like that just for consistency :)
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.
Thanks for the nice PR and please excuse the late review :/ Not much to comment, you thought of everything IISC 🥳
Note: | ||
You can only edit messages that the bot sent itself (i.e. of the ``bot.send_*`` family | ||
of methods) or channel posts, if the bot is an admin in that channel. However, this | ||
behaviour is undocumented and might be changed by Telegram. |
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.
If the text is mostly the same everywhere, I'd vote to keep it like that just for consistency :)
Check-list for PRs
.. versionadded:: NEXT.VERSION
,.. versionchanged:: NEXT.VERSION
,.. deprecated:: NEXT.VERSION
or ``.. versionremoved:: NEXT.VERSION` to the docstrings for user facing changes (for methods/class descriptions, arguments and attributes)__all__
sIf the PR contains API changes (otherwise, you can ignore this passage)Checked the Bot API specific sections of the Stability Policy
Created a PR to remove functionality deprecated in the previous Bot API release (see here)
New Classes
self._id_attrs
and corresponding documentation__init__
acceptsapi_kwargs
as keyword-onlyAdded New Shortcuts
telegram.Chat
&telegram.User
for all methods that acceptchat/user_id
telegram.Message
for all methods that acceptchat_id
andmessage_id
telegram.Message
shortcuts: Addedquote
argument if methods acceptreply_to_message_id
telegram.CallbackQuery
for all methods that accept eitherchat_id
andmessage_id
orinline_message_id
If Relevant
telegram.constants
and shortcuts to them as class variablestelegram.Message.effective_attachment
telegram.ext.ConversationHandler
_extbot.py
bot_methods.rst
README.rst
(including the badge) andtelegram.constants.BOT_API_VERSION_INFO
telegram.ext.ExtBot
for new methods that either accept areply_markup
in some form or have a return type that is/containstelegram.Message
API 9.1 Items:
I've skipped
parse_entity
utility methods for the classesInputChecklist[Task]
as they're implemented on the returned version of the classes, and don't make much sense here imo.