-
Notifications
You must be signed in to change notification settings - Fork 2.1k
Bot API 7.9 #2377
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
Bot API 7.9 #2377
Conversation
…ansactions involving paid media.
…ransactions involving paid media.
"Added the field until_date to the class ChatMemberMember for members with an active subscription." |
First two didn't require a commit either. |
@Badiboy any news on this |
Did not see in tonns of notifications. See now. |
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.
Everything except "name=name" proposal LGTM.
telebot/__init__.py
Outdated
:rtype: :class:`telebot.types.ChatInviteLink` | ||
""" | ||
return types.ChatInviteLink.de_json( | ||
apihelper.create_chat_subscription_invite_link(self.token, chat_id, subscription_period, subscription_price, name) |
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.
I propose to make optional parameters named.
... name=name
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 the name?
telebot/__init__.py
Outdated
:rtype: :class:`telebot.types.ChatInviteLink` | ||
""" | ||
return types.ChatInviteLink.de_json( | ||
apihelper.edit_chat_subscription_invite_link(self.token, chat_id, invite_link, name) |
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.
The same here.
name=name
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.
In ASYNC also in 2 places.
👍 |
Uh oh!
There was an error while loading. Please reload this page.