team21-movies-services / notify-service

Сервис по отправки уведомлений
0 stars 0 forks source link

Code review #63

Open BigDeepBlue opened 1 year ago

BigDeepBlue commented 1 year ago

Очень хорошо структурированный код 🔥 Проект завелся без проблем.

  1. Неужели это нужно в проде именно в таком виде? https://github.com/team21-movies-services/notify-service/blob/0c6dd80ef9c339bc37f5a4e132cb9a0e515176dc/src/celery_worker/backend/notify.py#L43-L46
  2. Вот этот Enum специально расположен отдельно от остальных enum-ов?
  3. Тестами сервис нотификации покрыт, мягко говоря, неубедительно 🤨
  4. Для работы с почтой в dev окружении рекомендую https://github.com/mailhog/MailHog. Очень удобный инструмент выступающий в качестве SMTP сервера и web интерфейсом для просмотра всех отправленных сообщений. Вот еще пример https://akrabat.com/using-mailhog-via-docker-for-testing-email/
Avis20 commented 1 year ago

Исправлено

BigDeepBlue commented 1 year ago

LGTM

BigDeepBlue commented 1 year ago

Наверняка знакомы, но на всякий случай https://python-dependency-injector.ets-labs.org/examples/index.html Это контейнер внедрения зависимостей, который вы реализовали самостоятельно.