Skip to content

fix: исправления по ревью PR #61#1

Closed
goduni wants to merge 27 commits intomasterfrom
fix/pr-61-review
Closed

fix: исправления по ревью PR #61#1
goduni wants to merge 27 commits intomasterfrom
fix/pr-61-review

Conversation

@goduni
Copy link
Copy Markdown
Owner

@goduni goduni commented Mar 16, 2026

Тест

K1rL3s and others added 27 commits February 27, 2026 12:50
- `CallbackQuery` как алиас на `MessageCallback`
- Проперти для вложений в `MessageBody`
- Ключ `"state"` c `FsmContext`
…achmentRequests | Attachments`'

Спасибо @ivan_kirpichnikov помог слава богу
- Изменил названия функций с телеграма на макс:
- `maxo.utils.link.create_telegram_link` -> `create_http_max_link`
- `maxo.utils.link.create_tg_link` -> `create_telegram_link`
Перенёс `CommandObject` из `maxo.types.command_object` в `maxo.routing.filters.command`
…= self.message_callback` в `Router` как подражание `aiogram`
Вероятность коллизии стала меньше, и длина айдишника стала 6 вместо 5-6
- ids.py: from mypy.types → from typing
- webhook engine: dump() → load() для десериализации
- formatting: защита от пустых аргументов в as_line/as_list
- FastAPI adapter: on_startup/on_shutdown подключены через add_event_handler
- user_repo: INSERT OR IGNORE + re-fetch для защиты от гонки
- bot.py: docstring-предупреждение о секретности token
- pyproject.toml: [tool.pytest] → [tool.pytest.ini_options]
- message facade: подстановка defaults.disable_link_preview
@goduni goduni closed this Mar 16, 2026
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.

2 participants