Conversation
Co-authored-by: Hinrich Mahler <22366557+Bibo-Joshi@users.noreply.github.com>
Co-authored-by: Bibo-Joshi <22366557+Bibo-Joshi@users.noreply.github.com>
hope i didn't interupt, Hinrich :)
| message_id: int, | ||
| chat_id: Optional[Union[int, str]] = None, | ||
| allow_sending_without_reply: ODVInput[bool] = DEFAULT_NONE, | ||
| checklist_task_id: Optional[int] = None, |
There was a problem hiding this comment.
this is a breaking change @aelkheir @Bibo-Joshi
should be added to the end of the parameter list
There was a problem hiding this comment.
Damn, we missed that. Maybe we should change it to make it non breaking. With the infinite retry bug we would then have two bugs for a bug fix release
There was a problem hiding this comment.
would changing the order of the argument again make it breaking again 🤔
we could just accept it was breaking and announce as such in the channel
There was a problem hiding this comment.
I think it's better to change it back to last position.
Either way we'll break users' code, we'll just be breaking a lesser amount of code assuming a lesser proportion of users updated since the last release
Closes #4910
Check-list for PRs
.. versionadded:: NEXT.VERSION,.. versionchanged:: NEXT.VERSION,.. deprecated:: NEXT.VERSIONor ``.. versionremoved:: NEXT.VERSION` to the docstrings for user facing changes (for methods/class descriptions, arguments and attributes)AUTHORS.rst(optional)__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_attrsand corresponding documentation__init__acceptsapi_kwargsas keyword-onlyAdded New Shortcuts
telegram.Chat&telegram.Userfor all methods that acceptchat/user_idtelegram.Messagefor all methods that acceptchat_idandmessage_idtelegram.Messageshortcuts: Addedquoteargument if methods acceptreply_to_message_idtelegram.CallbackQueryfor all methods that accept eitherchat_idandmessage_idorinline_message_idIf Relevant
telegram.constantsand shortcuts to them as class variablestelegram.Message.effective_attachmenttelegram.ext.ConversationHandler_extbot.pybot_methods.rstREADME.rst(including the badge) andtelegram.constants.BOT_API_VERSION_INFOtelegram.ext.ExtBotfor new methods that either accept areply_markupin some form or have a return type that is/containstelegram.Message