feat: payment checker, history cache, webhook support, CLI - #30
feat: payment checker, history cache, webhook support, CLI#30CLoneaps wants to merge 14 commits into
Conversation
|
ты забыл свой токен убрать |
|
Убрал, спасибо! |
| @@ -1,8 +1,11 @@ | |||
| from yoomoney._async_client import AsyncClient | |||
| from yoomoney.cache.cache import BaseCache, JSONCache, SQLiteCache # ← добавить | |||
There was a problem hiding this comment.
почисти код
перечитай каждое изменение и подумай нужно ли оно
комментарии и тд
8108e1e to
1eb2274
Compare
040b5f1 to
5355688
Compare
e6e582f to
d0d479c
Compare
213c47b to
53249cd
Compare
|
Мне кажется, что кэш это всё-таки логика приложения, а не задача API-клиента. Если реализуете кэширование внутри, то неявно навязываете разработчику конкретный подход. А кому-то нужен Redis, кому-то файл, а кто-то вообще хочет кэшировать на уровне Nginx Консольная утилита штука полезная, но для основной библиотеки это лишний вес. Большинство будут подключать пакет в веб-приложения, где CLI не нужен, а зависимости тянуть придётся всем |
Согласен с обоими замечаниями. По кэшу ты прав, это логика приложения, не клиента. Уберу из ядра. По CLI тоже согласен, лишний вес для большинства не будет почти юзаться. Уберу, если будет спрос можно вынести в отдельный пакет позже. Пересоберу PR без этих моментов когда будет время сегодня. |
- Remove yoomoney/cache/ module (SQLiteCache, JSONCache, BaseCache) - Remove yoomoney/cli.py and CLI entry point - Remove examples/cache_history.py - Update __init__.py: drop cache imports from __all__ - Update pyproject.toml: remove [project.scripts], keywords, description - Update README.rst and README_RU.rst: remove History cache and CLI sections
- Remove yoomoney/cache/ module (SQLiteCache, JSONCache, BaseCache) - Remove yoomoney/cli.py and CLI entry point - Remove examples/cache_history.py - Update __init__.py: drop cache imports from __all__ - Update pyproject.toml: remove [project.scripts], keywords, description - Update README.rst and README_RU.rst: remove History cache and CLI sections
|
Убрал кэш и CLI из ядра библиотеки. Удалён модуль yoomoney/cache/ (SQLiteCache, JSONCache, BaseCache) |
3 participants
No description provided.