Fix comment_to for a group of messages #4120
Merged
Conversation
This file contains 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
|
Are we sure it will always be |
|
fair enough, fixed |
|
Thanks. |
vanutp
pushed a commit
to vanutp/Telethon
that referenced
this pull request
Sep 28, 2023
New-dev0
added a commit
to TeamUltroid/Telethon
that referenced
this pull request
Dec 11, 2023
* Change bug report template to an issue form Lifted from Tachiyomi and adapted for Telethon. See: https://github.com/tachiyomiorg/tachiyomi/blob/master/.github/ISSUE_TEMPLATE/report_issue.yml * Change feature request template to an issue form * Add dedicated form for documentation issues * Update some fields in the GH issue template * Fix comment_to for a group of messages (LonamiWebs#4120) * Clarify OS field in bug report template * Handle CancelledError inside mtprotosender recv loop * Add missing formatting arg in logging call Noticed in LonamiWebs#4123. * Document more RPC errors (LonamiWebs#4127) * Support most usernames in VALID_USERNAME_RE See LonamiWebs#4128. * Fix lack of support for anon channel restrictions (LonamiWebs#4130) * Improve error message when trying to delete inline messages Closes LonamiWebs#4129. * Change html.unparse logic to mimic markdown's It was overcomplicated and had some subtle bugs. Closes LonamiWebs#4133. * Update to layer 160 * Fix reply_to can be optional * Fix _get_thumb failed when document had no thumbs * Remove client-side checks when editing permissions The server should instead fail with proper RPC errors, as the rules could change any time (and the local checks get out of date). * Handle sqlite3.OperationalError in update loop * Handle FloodWaitError in update loop 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. * Don't attempt thumb download if there is no thumb * Disable blank issues in GitHub (LonamiWebs#4157) * Bump to v1.29.0 * Fix handling of UpdateShortSentMessage * Improve image compression heuristics and algorithm used (LonamiWebs#4158) * Fix reply_to when sending albums * Sort tlobjects before generating their listing (LonamiWebs#4163) * Don't treat asyncio.IncompleteReadError as unhandled The library will behave the same, but the log severity is lowered. * Except all types of timeout error Closes LonamiWebs#4172. * Return marked ID from MemorySession.get_entity_rows_by_id (LonamiWebs#4177) Otherwise the unpacking done later won't work. * Fix `is_inline` check for `KeyboardButtonWebView` (LonamiWebs#4183) * Make MessageBox trace logs more useful * Fix getting_diff_for with empty set was being spammed Because the above check used >= but the inner check >. * Further improve MessageBox trace logging * Only update seq if pts changed This solves UpdateChatParticipant being missed after UpdateChat, which seems to reliable occur when a bot is in a Chat that gets deleted. * Change apply_deadlines_reset micro-optimization No need for buffer reuse in Python. It simply complicates the code. And even then it was not as optimal as it could. * Update to layer 161 Closes LonamiWebs#4184. * Fix incorrect param type in apply_channel_difference (LonamiWebs#4185) * Fix date empty (LonamiWebs#4191) * Remove client-side check in message.edit (LonamiWebs#4195) Fixes LonamiWebs#4193. * Update to layer 162 * Bump to v1.30 * Fix generator for pypy (LonamiWebs#4198) * Fix invalid date type in UpdateShort * Fix date empty when getting difference * Fixed sorting of markup entities with the same offsets (LonamiWebs#4201) * Remove uses of imghdr It's deprecated. Closes LonamiWebs#4207. * Update to layer 164 * Update to layer 165 * Update to v1.31 * Fix file name could be lost when uploading files Leading to invalid extension when sending photos. * Update to layer 166 * Fix init of custom Draft after layer update * Bump to v1.32 * Add PR template mentioning v1 is feature-frozen Should prevent efforts like LonamiWebs#4244 going to waste in the future. * Force filename with JPG extension after resizing Old name does not matter, since we just encoded it as JPEG * Retry on TimedOutError (LonamiWebs#4255) * Conditional webbrowser import (LonamiWebs#4261) * Update to layer 167 * Bump to v1.33 * Prioritise closing tags when sorting tags * Fix ordering of nested entities * Bump to v1.33.1 * 'Refactored by Sourcery' --------- Co-authored-by: novenary <[email protected]> Co-authored-by: Lonami <[email protected]> Co-authored-by: iamilya <[email protected]> Co-authored-by: Devesh Pal <[email protected]> Co-authored-by: rozha <[email protected]> Co-authored-by: novenary <[email protected]> Co-authored-by: Nick80835 <[email protected]> Co-authored-by: Bernhard M. Wiedemann <[email protected]> Co-authored-by: Kacnep89 <[email protected]> Co-authored-by: misuzu <[email protected]> Co-authored-by: Shubham Kumar <[email protected]> Co-authored-by: Non <[email protected]> Co-authored-by: Dingyuan Wang <[email protected]> Co-authored-by: Alexander Goryushkin <[email protected]> Co-authored-by: mario-ttide <[email protected]> Co-authored-by: Balázs Triszka <[email protected]> Co-authored-by: udf <[email protected]> Co-authored-by: Sourcery AI <>
New-dev0
added a commit
to TeamUltroid/Telethon
that referenced
this pull request
Dec 12, 2023
* Change bug report template to an issue form Lifted from Tachiyomi and adapted for Telethon. See: https://github.com/tachiyomiorg/tachiyomi/blob/master/.github/ISSUE_TEMPLATE/report_issue.yml * Change feature request template to an issue form * Add dedicated form for documentation issues * Update some fields in the GH issue template * Fix comment_to for a group of messages (LonamiWebs#4120) * Clarify OS field in bug report template * Handle CancelledError inside mtprotosender recv loop * Add missing formatting arg in logging call Noticed in LonamiWebs#4123. * Document more RPC errors (LonamiWebs#4127) * Support most usernames in VALID_USERNAME_RE See LonamiWebs#4128. * Fix lack of support for anon channel restrictions (LonamiWebs#4130) * Improve error message when trying to delete inline messages Closes LonamiWebs#4129. * Change html.unparse logic to mimic markdown's It was overcomplicated and had some subtle bugs. Closes LonamiWebs#4133. * Update to layer 160 * Fix reply_to can be optional * Fix _get_thumb failed when document had no thumbs * Remove client-side checks when editing permissions The server should instead fail with proper RPC errors, as the rules could change any time (and the local checks get out of date). * Handle sqlite3.OperationalError in update loop * Handle FloodWaitError in update loop 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. * Don't attempt thumb download if there is no thumb * Disable blank issues in GitHub (LonamiWebs#4157) * Bump to v1.29.0 * Fix handling of UpdateShortSentMessage * Improve image compression heuristics and algorithm used (LonamiWebs#4158) * Fix reply_to when sending albums * Sort tlobjects before generating their listing (LonamiWebs#4163) * Don't treat asyncio.IncompleteReadError as unhandled The library will behave the same, but the log severity is lowered. * Except all types of timeout error Closes LonamiWebs#4172. * Return marked ID from MemorySession.get_entity_rows_by_id (LonamiWebs#4177) Otherwise the unpacking done later won't work. * Fix `is_inline` check for `KeyboardButtonWebView` (LonamiWebs#4183) * Make MessageBox trace logs more useful * Fix getting_diff_for with empty set was being spammed Because the above check used >= but the inner check >. * Further improve MessageBox trace logging * Only update seq if pts changed This solves UpdateChatParticipant being missed after UpdateChat, which seems to reliable occur when a bot is in a Chat that gets deleted. * Change apply_deadlines_reset micro-optimization No need for buffer reuse in Python. It simply complicates the code. And even then it was not as optimal as it could. * Update to layer 161 Closes LonamiWebs#4184. * Fix incorrect param type in apply_channel_difference (LonamiWebs#4185) * Fix date empty (LonamiWebs#4191) * Remove client-side check in message.edit (LonamiWebs#4195) Fixes LonamiWebs#4193. * Update to layer 162 * Bump to v1.30 * Fix generator for pypy (LonamiWebs#4198) * Fix invalid date type in UpdateShort * Fix date empty when getting difference * Fixed sorting of markup entities with the same offsets (LonamiWebs#4201) * Remove uses of imghdr It's deprecated. Closes LonamiWebs#4207. * Update to layer 164 * Update to layer 165 * Update to v1.31 * Fix file name could be lost when uploading files Leading to invalid extension when sending photos. * Update to layer 166 * Fix init of custom Draft after layer update * Bump to v1.32 * Add PR template mentioning v1 is feature-frozen Should prevent efforts like LonamiWebs#4244 going to waste in the future. * Force filename with JPG extension after resizing Old name does not matter, since we just encoded it as JPEG * Retry on TimedOutError (LonamiWebs#4255) * Conditional webbrowser import (LonamiWebs#4261) * Update to layer 167 * Bump to v1.33 * Prioritise closing tags when sorting tags * Fix ordering of nested entities * Bump to v1.33.1 * 'Refactored by Sourcery' --------- Co-authored-by: novenary <[email protected]> Co-authored-by: Lonami <[email protected]> Co-authored-by: iamilya <[email protected]> Co-authored-by: Devesh Pal <[email protected]> Co-authored-by: rozha <[email protected]> Co-authored-by: novenary <[email protected]> Co-authored-by: Nick80835 <[email protected]> Co-authored-by: Bernhard M. Wiedemann <[email protected]> Co-authored-by: Kacnep89 <[email protected]> Co-authored-by: misuzu <[email protected]> Co-authored-by: Shubham Kumar <[email protected]> Co-authored-by: Non <[email protected]> Co-authored-by: Dingyuan Wang <[email protected]> Co-authored-by: Alexander Goryushkin <[email protected]> Co-authored-by: mario-ttide <[email protected]> Co-authored-by: Balázs Triszka <[email protected]> Co-authored-by: udf <[email protected]> Co-authored-by: Sourcery AI <>
New-dev0
added a commit
to TeamUltroid/Telethon
that referenced
this pull request
May 13, 2024
* Change bug report template to an issue form Lifted from Tachiyomi and adapted for Telethon. See: https://github.com/tachiyomiorg/tachiyomi/blob/master/.github/ISSUE_TEMPLATE/report_issue.yml * Change feature request template to an issue form * Add dedicated form for documentation issues * Update some fields in the GH issue template * Fix comment_to for a group of messages (LonamiWebs#4120) * Clarify OS field in bug report template * Handle CancelledError inside mtprotosender recv loop * Add missing formatting arg in logging call Noticed in LonamiWebs#4123. * Document more RPC errors (LonamiWebs#4127) * Support most usernames in VALID_USERNAME_RE See LonamiWebs#4128. * Fix lack of support for anon channel restrictions (LonamiWebs#4130) * Improve error message when trying to delete inline messages Closes LonamiWebs#4129. * Change html.unparse logic to mimic markdown's It was overcomplicated and had some subtle bugs. Closes LonamiWebs#4133. * Update to layer 160 * Fix reply_to can be optional * Fix _get_thumb failed when document had no thumbs * Remove client-side checks when editing permissions The server should instead fail with proper RPC errors, as the rules could change any time (and the local checks get out of date). * Handle sqlite3.OperationalError in update loop * Handle FloodWaitError in update loop 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. * Don't attempt thumb download if there is no thumb * Disable blank issues in GitHub (LonamiWebs#4157) * Bump to v1.29.0 * Fix handling of UpdateShortSentMessage * Improve image compression heuristics and algorithm used (LonamiWebs#4158) * Fix reply_to when sending albums * Sort tlobjects before generating their listing (LonamiWebs#4163) * Don't treat asyncio.IncompleteReadError as unhandled The library will behave the same, but the log severity is lowered. * Except all types of timeout error Closes LonamiWebs#4172. * Return marked ID from MemorySession.get_entity_rows_by_id (LonamiWebs#4177) Otherwise the unpacking done later won't work. * Fix `is_inline` check for `KeyboardButtonWebView` (LonamiWebs#4183) * Make MessageBox trace logs more useful * Fix getting_diff_for with empty set was being spammed Because the above check used >= but the inner check >. * Further improve MessageBox trace logging * Only update seq if pts changed This solves UpdateChatParticipant being missed after UpdateChat, which seems to reliable occur when a bot is in a Chat that gets deleted. * Change apply_deadlines_reset micro-optimization No need for buffer reuse in Python. It simply complicates the code. And even then it was not as optimal as it could. * Update to layer 161 Closes LonamiWebs#4184. * Fix incorrect param type in apply_channel_difference (LonamiWebs#4185) * Fix date empty (LonamiWebs#4191) * Remove client-side check in message.edit (LonamiWebs#4195) Fixes LonamiWebs#4193. * Update to layer 162 * Bump to v1.30 * Fix generator for pypy (LonamiWebs#4198) * Fix invalid date type in UpdateShort * Fix date empty when getting difference * Fixed sorting of markup entities with the same offsets (LonamiWebs#4201) * Remove uses of imghdr It's deprecated. Closes LonamiWebs#4207. * Update to layer 164 * Update to layer 165 * Update to v1.31 * Fix file name could be lost when uploading files Leading to invalid extension when sending photos. * Update to layer 166 * Fix init of custom Draft after layer update * Bump to v1.32 * Add PR template mentioning v1 is feature-frozen Should prevent efforts like LonamiWebs#4244 going to waste in the future. * Force filename with JPG extension after resizing Old name does not matter, since we just encoded it as JPEG * Retry on TimedOutError (LonamiWebs#4255) * Conditional webbrowser import (LonamiWebs#4261) * Update to layer 167 * Bump to v1.33 * Prioritise closing tags when sorting tags * Fix ordering of nested entities * Bump to v1.33.1 * Fix typo in documentation example (LonamiWebs#4277) * Fix restriction_reason type hint (LonamiWebs#4282) * Update to layer 169 * Update to layer 170 * Add missing saved_peer_id parameter to Message * Add new config file for readthedocs * Update to layer 171 * Add reply_to_chat and reply_to_sender in Message (LonamiWebs#4300) * Update to layer 172 * 'Refactored by Sourcery' --------- Co-authored-by: novenary <[email protected]> Co-authored-by: Lonami <[email protected]> Co-authored-by: iamilya <[email protected]> Co-authored-by: Devesh Pal <[email protected]> Co-authored-by: rozha <[email protected]> Co-authored-by: novenary <[email protected]> Co-authored-by: Nick80835 <[email protected]> Co-authored-by: Bernhard M. Wiedemann <[email protected]> Co-authored-by: Kacnep89 <[email protected]> Co-authored-by: misuzu <[email protected]> Co-authored-by: Shubham Kumar <[email protected]> Co-authored-by: Non <[email protected]> Co-authored-by: Dingyuan Wang <[email protected]> Co-authored-by: Alexander Goryushkin <[email protected]> Co-authored-by: mario-ttide <[email protected]> Co-authored-by: Balázs Triszka <[email protected]> Co-authored-by: udf <[email protected]> Co-authored-by: Allen Calderwood <[email protected]> Co-authored-by: Jahongir Qurbonov <[email protected]> Co-authored-by: exovoq <[email protected]> Co-authored-by: Sourcery AI <>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
GetDiscussionMessageRequestreturns a list of messages in reverse order (which is crucial in case if it is a group of photos/docs/etc), you need to reply to the first message so that a comment is displayed correctly.