-
Notifications
You must be signed in to change notification settings - Fork 6k
Labels
ℹ️ good-first-issueinformation: good-first-issueinformation: good-first-issue
Milestone
Description
Current situation
Telegram currently offers attribute message_id to Message. But this is inconsistent with the rest of their API, e.g. Chat.id, User.id, InlineQuery.id, etc. We can offer a convenience property Message.id which will make it more consistent and make it less prone to errors, e.g. when someone forgets it's message_id and not id.
Proposed change
Add the property id to Message which just returns self.message_id. A similar example can be found here.
A PR for this issue should
- Add a docstring along with a
.. versionadded:: 20.0directive to the function. - Add an unit test to
test_message.pythat ensures that this works as expected and is the same asMessage.message_id.
Additional Info
We have a contribution guide
Metadata
Metadata
Assignees
Labels
ℹ️ good-first-issueinformation: good-first-issueinformation: good-first-issue