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

Sync Changes #23

Open
wants to merge 139 commits into
base: ultroid
Choose a base branch
from
Open

Sync Changes #23

wants to merge 139 commits into from

Conversation

New-dev0
Copy link
Member

No description provided.

9ary and others added 30 commits May 29, 2023 21:29
It was overcomplicated and had some subtle bugs.
Closes #4133.
The server should instead fail with proper RPC errors,
as the rules could change any time (and the local checks
get out of date).
Likely temporary server issues, since getDifference should
realistically not fail with flood waits. In any case,
stopping early until the problem is resolved is the correct
approach.
The library will behave the same, but the log severity is lowered.
Otherwise the unpacking done later won't work.
EntireMusic and others added 21 commits October 5, 2024 11:13
The sign_up method was deprecated in
07a7a8b and removed from the
documentation, but since the method technically still exists (even
though it immediately raises ValueError), that broke a test.
Introduced on accident by #4475.
This fixes image compression with mode "P" (potentially others) which is necessary as the server has erroneous alpha color with some types of images (mode "P" for example). This also properly applies the background argument that may be passed to _resize_photo_if_needed by always compressing images with alpha regardless of whether the server will compress the image for us.
drop_author is already supported but is undocumented. drop_media_captions for consistency with drop_author being implemented.
This also allows utils.get_message_id to get the ID of InputMessageID.
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.