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

ACC-72 Привести в порядок Admin API #43

Merged
merged 4 commits into from
Jun 30, 2021
Merged

Conversation

Drevoed
Copy link
Collaborator

@Drevoed Drevoed commented Jun 27, 2021

Помимо основной задачи этот пулл реквест так же включает в себя:

  • Замену фасада для логов (log -> tracing)
  • Функционал инжекта сервисов в App
  • Перенос привязанных к транспортному фрейму сервисов в core
  • Отдельный пакет app
  • Оптимизация путем убирания (блокирующего!) RwLock на App (зачем-то висели в трейтах &mut self)

Инжект сервисов

Теперь App не привязан к конкретным сервисам дженериками, вместо этого необходимые трейт обжекты сервисов создаются и инсертятся на этапе инициализации приложения.

image
image
image

@linear
Copy link

linear bot commented Jun 27, 2021

ACC-72 Привести в порядок Admin API

  • Привести структуру крейта в соответствие с остальными api
  • Обновить зависимости

@Drevoed Drevoed requested a review from sergeysova June 27, 2021 17:49
@Drevoed Drevoed changed the title Acc 72 ACC-72 Привести в порядок Admin API Jun 30, 2021
@Drevoed Drevoed self-assigned this Jun 30, 2021
@Drevoed Drevoed marked this pull request as ready for review June 30, 2021 21:04
@Drevoed Drevoed merged commit 73ae7d5 into master Jun 30, 2021
@Drevoed Drevoed deleted the ACC-72 branch June 30, 2021 21:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants