Skip to content

Fixes media-messages with caption #52 #53

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

SnipGhost
Copy link

  • Сделал небольшое костыльное исправление для SkipDuplicateMessageHandler - теперь там не выбрасывается KeyError при отсуствии в event.data поля text. Процесс dispatching-а вроде не ломает.
  • И устарнил основную проблему с фильтрами медиа при наличии подписей, описанную в Проблема с фильтрами медиа при наличии подписей #52 - теперь сообщения с подписями ловятся в фильтром MessageFilter.

Если считаете, что новое поведение стоит унести в новый класс - уточните название, внесу необходимые изменения.

@SnipGhost
Copy link
Author

К комиту fbc94fa - может стоит сравнивать messageID из кэша и messageID, а не тексты? Кажется тут тогда еще может косячить, если текст у сообщений совпадает...

@SnipGhost SnipGhost changed the title Fixes #52 Fixes media-messages with caption #52 Aug 7, 2025
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.

1 participant