Skip to content
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

Избавиться от интерфейсной модели #30

Open
3 tasks
viribus-issue-bot opened this issue Aug 23, 2024 · 0 comments · May be fixed by #32
Open
3 tasks

Избавиться от интерфейсной модели #30

viribus-issue-bot opened this issue Aug 23, 2024 · 0 comments · May be fixed by #32
Assignees

Comments

@viribus-issue-bot
Copy link

Issue open by Roman Dyakov via telegram message.

#рефакторинг №1

  • Удалить алерты. Сейчас существует функционал Alert, который ничего не делает и является формально историей рассылки с странным интерфейсом. Удалим пока связанные с ним ручки, алертить и менять алерты все равно пользователям нельзя

  • Удалить интерфейсы и классы сервисов. Сейчас есть интерфейсы вида AlertServiceInterface в base и классы class PgAlertService(AlertServiceInterface) в конкретных сервисах. Это кажется избыточным, т.к. использовать это невозможно. Лучше оставить просто функции вида def create_alert в тех же расположениях.

  • Удалить асинхронность. Функции в сервисах асинхронные, но далее это никак не используется. Нужно удалить все связанные асинки и соответствующие эвэйты.

@github-project-automation github-project-automation bot moved this to Backlog in Viribus Team Aug 23, 2024
@Wudext Wudext linked a pull request Oct 9, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Backlog
Development

Successfully merging a pull request may close this issue.

2 participants