Upd tg-bot to v20 #8
Conversation
filters.py
Outdated
| @@ -11,7 +11,7 @@ class HasNoValidPreviousMessages(MessageFilter): | |||
| MIN_PREVIOUS_MESSAGES_COUNT = 3 | |||
|
|
|||
| def filter(self, message: Message) -> bool: | |||
| if not DB_ENABLED(): | |||
| if not DB_ENABLED() or not message.from_user: | |||
There was a problem hiding this comment.
А в каких случаях у сообщения может не быть from_user?
There was a problem hiding this comment.
Тут и ниже где проверка if message - в библиотеке обновились немного типы, и теперь from_user стал Optional и update.message/edited_message тоже Optional. Mypy это подсветил и я добавил доп проверки
bot.py
Outdated
| message_id=message.message_id, | ||
| chat_id=message.chat_id, | ||
| ) | ||
| if message: |
There was a problem hiding this comment.
А в каких случаях message может быть False?
There was a problem hiding this comment.
Я тут просто на наличие проверяю переменую. Критично ли писать так, а не is not None?
There was a problem hiding this comment.
Мой вопрос был в том, в каких случаях этой переменной может не быть и почему?
There was a problem hiding this comment.
А, ну тут может быть вполне, что в Update пришло не сообщение, а что-то из channel_post, poll, poll_answer etc
bot.py
Outdated
| @@ -33,7 +23,6 @@ def log_message(message: Message, action: Optional[str] = ''): | |||
| text=message.text or '', | |||
| meta={ | |||
| 'tags': [ | |||
| *rekognition.get_labels(image_url=get_profile_picture(message)), | |||
| *text.Labels(message.text)(), | |||
There was a problem hiding this comment.
Конструкция [*text.Labels(message.text)()] стала выглядеть довольно странно. Почему бы не сделать просто text.Labels(message.text)()?
Обновил версию библиотеки для бота до актуальной + перенес конфиги к pyproject.toml (#7)
Саму логику нигде никак не менял. Все изменения какие есть - только в следствие изменения апи библиотек на совместимые. После обновления затестил бота - работает так же как и до этого, ничего не сломалось.