Skip to content

Conversation

@Poolitzer
Copy link
Member

@Poolitzer Poolitzer commented Mar 9, 2021

When merged, this closes #2423.

Open questions:

  • Should two ChatInviteLinks be equal, when the invite link and creator are equal, or in some other scenario?

Checklist for PRs

  • Added .. versionadded:: version, .. versionchanged:: version or .. deprecated:: version to the docstrings for user facing changes (for methods/class descriptions, arguments and attributes)

If the PR contains API changes (otherwise, you can delete this passage)

  • New classes:

    • Added self._id_attrs and corresponding documentation
    • __init__ accepts **_kwargs
  • Added new / updated shortcuts:

    • In Chat & User for all methods that accept chat/user_id
    • [ ] In Message for all methods that accept chat_id and message_id
    • [ ] For new Message shortcuts: Added quote argument if methods accepts reply_to_message_id
    • [ ] In CallbackQuery for all methods that accept either chat_id and message_id or inline_message_id
  • If relevant:

    • Added new constants at telegram.constants and shortcuts to them as class variables
    • Added new handlers for new update types
    • Added new filters for new message (sub)types
    • Added or updated documentation for the changed class(es) and/or method(s)
    • Updated the Bot API version number in all places in README.rst and README_RAW.rst, including the badge

Active docs build is at https://python-telegram-bot.readthedocs.io/en/api_5.1/

@Poolitzer Poolitzer closed this Mar 9, 2021
@Poolitzer Poolitzer reopened this Mar 9, 2021
@Bibo-Joshi Bibo-Joshi added the ⚙️ bot-api affected functionality: bot-api label Mar 9, 2021
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey! Looks like you edited the (dev) requirements or the pre-commit hooks. I'm just a friendly reminder too keep the pre-commit hook versions in sync with the dev requirements and the additional dependencies for the hooks in sync with the requirements :)

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey there. Relax, I am just a little warning for the maintainers to release directly after merging your PR, otherwise we have broken examples and people might get confused :)

@python-telegram-bot python-telegram-bot deleted a comment from Poolitzer Mar 9, 2021
@Bibo-Joshi
Copy link
Member

ChatMemberHandler is up for discussion. Adding two handlers MyChatMemberHandler and ChatMemberHandler seemed a bit extensive to me, so thought that this could be a feasible mid way.

Copy link
Member

@Bibo-Joshi Bibo-Joshi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Poolitzer : We'll need new shortcuts for Chat.do_stuff_with_invite_links

@starry69 we'll need filters for the new message types

Signed-off-by: starry69 <starry369126@outlook.com>
Signed-off-by: starry69 <starry369126@outlook.com>
@starry-shivam
Copy link
Member

@Poolitzer : We'll need new shortcuts for Chat.do_stuff_with_invite_links

@starry69 we'll need filters for the new message types

Yea gonna do in evening

Copy link
Member

@harshil21 harshil21 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

did another general review

@Bibo-Joshi Bibo-Joshi added this to the v13.4 milestone Mar 11, 2021
starry-shivam and others added 6 commits March 12, 2021 17:25
Signed-off-by: starry69 <starry369126@outlook.com>
Signed-off-by: starry69 <starry369126@outlook.com>
Signed-off-by: starry69 <starry369126@outlook.com>
change place of version adding, added obj:True as doc string, changing how member limit is initiated
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey! Looks like you edited README.rst or README_RAW.rst. I'm just a friendly reminder to apply relevant changes to both of those files :)

* add bot api version in constants

Signed-off-by: starry69 <starry369126@outlook.com>

* addressing review

Signed-off-by: starry69 <starry369126@outlook.com>

* add versioning directive

Co-authored-by: Bibo-Joshi <hinrich.mahler@freenet.de>
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey! Looks like you edited README.rst or README_RAW.rst. I'm just a friendly reminder to apply relevant changes to both of those files :)

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey! Looks like you edited README.rst or README_RAW.rst. I'm just a friendly reminder to apply relevant changes to both of those files :)

@Bibo-Joshi Bibo-Joshi merged commit ac02bce into master Mar 14, 2021
@Bibo-Joshi Bibo-Joshi deleted the api_5.1 branch March 14, 2021 15:41
@Bibo-Joshi Bibo-Joshi mentioned this pull request Mar 14, 2021
@CustomIcon
Copy link

edits README
github-actions bot: 🔫

@python-telegram-bot python-telegram-bot locked as resolved and limited conversation to collaborators Mar 14, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

⚙️ bot-api affected functionality: bot-api

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEATURE] API 5.1 support

6 participants