Skip to content
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 8.2 full support #2440

Merged
merged 2 commits into from
Jan 3, 2025
Merged

Conversation

coder2020official
Copy link
Collaborator

@coder2020official coder2020official commented Jan 2, 2025

January 1, 2025
Bot API 8.2

@coder2020official
Copy link
Collaborator Author

@Badiboy done with the bot api, you can start reviewing

@@ -2271,12 +2301,6 @@ def convert_input_media_array(array):
if media_dict['media'].startswith('attach://'):
key = media_dict['media'].replace('attach://', '')
files[key] = input_media.media
Copy link
Collaborator

Choose a reason for hiding this comment

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

What is the reason for removing this code?

:return: Returns True on success.
:rtype: :obj:`bool`
"""
return await asyncio_helper.verify_user(self.token, user_id, custom_description)
Copy link
Collaborator

Choose a reason for hiding this comment

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

May be it's time to start sending optional parameters as named?

@@ -2243,12 +2269,6 @@ async def convert_input_media_array(array):
if media_dict['media'].startswith('attach://'):
key = media_dict['media'].replace('attach://', '')
files[key] = input_media.media
Copy link
Collaborator

Choose a reason for hiding this comment

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

The same here: what is the reason for removing this code?

@@ -1,3 +1,3 @@
# Versions should comply with PEP440.
# This line is parsed in setup.py:
__version__ = '4.25.0'
__version__ = '4.24.0'
Copy link
Collaborator

Choose a reason for hiding this comment

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

Version downgraded.

@Badiboy
Copy link
Collaborator

Badiboy commented Jan 3, 2025

Looks like you made PR not from the recent version...

@coder2020official
Copy link
Collaborator Author

@Badiboy sorry for that, it did not somehow show up on the changelog for me.
Fixed all addressed issues

@Badiboy
Copy link
Collaborator

Badiboy commented Jan 3, 2025

Thank you!

Release?

@Badiboy Badiboy merged commit a9d4e8b into eternnoir:master Jan 3, 2025
7 checks passed
@coder2020official
Copy link
Collaborator Author

Yes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants