-
Notifications
You must be signed in to change notification settings - Fork 5.1k
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
Add missing shortcuts #2043
Add missing shortcuts #2043
Conversation
Agree with that |
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.
Good PR, I cant believe we missed .pin()
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.
looks perfect now
Turns out, there where a lot more than I expected.
Chat
already has a methodsend_action
. I addedsend_chat_action
for completeness and both toUser
.TL;DR: Both. Both is good.
I guess the logic was to skip the
chat
similarly toMessage.edit_text
as shortcut forBot.edit_message_text
, but thechat
insend_chat_action
acutally comes from the class nameChatAction
… But having both doesn't hurt.Also, one could think of adding shortcuts to
ChosenInlineResult
, which has aninline_message_id
attribute, but IMHO that doesn't really make sense. It would only be needed, if one would edit the message right after it's posted (why, though?). In all other cases one usually has only theinline_message_id
and not theCIR
.Closes #2034