Update to resolve Issue with outdated _convert_entities
function in copy_message
method
#2240
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.
Description
I identified an issue with the outdated function
_convert_entities
used in both the sync and async versions of thecopy_message
method. This function was not functioning correctly, resulting in invalid entities. While all other methods were using a more updated method for converting entities, this particular method was still relying on_convert_entities
.To address this issue, I have removed
_convert_entities
and replaced it with more betterjson.dumps(types.MessageEntity.to_list_of_dicts(...))
. This change ensures that the entities are properly converted and are validPython version:
3.12
OS:
windows 10
Checklist: