Merged
Conversation
Poolitzer
reviewed
Jun 28, 2020
| days or less than 30 seconds from the current time, they are considered to be | ||
| restricted forever. | ||
| permissions (:class:`telegram.ChatPermissions`): New user permissions. | ||
| permissions (:class:`telegram.ChatPermissions`): A JSON-serialized object for new user |
Member
There was a problem hiding this comment.
But uhm, in our context, it would be a JSON- deserialized Telegram class/object. Since we convert it from the string to our own object.
Member
Author
There was a problem hiding this comment.
Just followed the convention to copy official docs. We have a bunch of similar docstrings … I'm fairly sure that passing json data would indeed also work
* Fix PicklePersistence.flush() with only bot_data (#2017) * Update pylint in pre-commit to fix CI (#2018) * Add Filters.via_bot (#2009) * feat: via_bot filter also fixing a small mistake in the empty parameter of the user filter and improve docs slightly * fix: forgot to set via_bot to None * fix: redoing subclassing to copy paste solution * Cosmetic changes Co-authored-by: Hinrich Mahler <hinrich.mahler@freenet.de> * Update CHANGES.rst Fixed Typo Co-authored-by: Bibo-Joshi <hinrich.mahler@freenet.de> Co-authored-by: Poolitzer <25934244+Poolitzer@users.noreply.github.com>
Replaced RegexHandler with MessageHandler, since the former is deprecated
Merged
True, False, None are now rendered with ``bool`` wherever they weren't in telegram and telegram.ext classes.
As usual, docs were cross-checked with official tg api docs.
As usual, docs were cross-checked with official tg api docs.
Mostly just added hyperlinks. And fixed message length doc. As usual, docs were cross-checked with official tg api docs.
Didn't bother changing too much since this seems to be a custom implementation.
As usual, cross-checked with official bot api docs.
Few tiny other fixes too.
Few tiny other doc fixes too.
Lot of doc fixes
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
restrict_chat_membersargumentchat_permissionfollowing official API docssend_invoicesargumentprovider_datafollowing official API docs