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

Добавление скрипта для приглашения первого супер-админа #637

Merged
merged 8 commits into from
Sep 20, 2024

Conversation

Genek91
Copy link
Collaborator

@Genek91 Genek91 commented Aug 16, 2024

Что сделано

Добавил скрипт для приглашения первого супер-админа после запуска приложения.

Как тестировал

Проверил, что при старте создаётся токен и приходит письмо с приглашением. После регистрации и перезапуска приложения, новый токен не создаётся и письмо не отправляется, согласно условию задачи.

src/api/utils.py Outdated Show resolved Hide resolved
src/api/utils.py Outdated Show resolved Hide resolved
src/api/utils.py Outdated Show resolved Hide resolved
@Xalgina
Copy link

Xalgina commented Aug 19, 2024

Протестировано. ОК

@Xalgina Xalgina self-requested a review August 19, 2024 18:48
Copy link

@Xalgina Xalgina left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ОК

@gorskyolga
Copy link
Collaborator

PR не проходит проверку "Check app" из-за отсутствия настроек почтового сервера в .github/workflows/healthcheck.yaml. Можно было бы их туда и передавать, но аналогичная проблема возникает при запуске бота локально без этих настроек. Если не прописывать настройки почтового сервера в .env, то приложение не будет запускаться локально.

Чтобы у разработчиков осталась возможность запускать бот локально без настроек почтового сервера, нужно в create_token_for_super_user перед инициализацией EmailProvider и отправкой письма предусмотреть проверку, что настройки почтового сервера заданы.
Можно проверять, что в settings переменные MAIL_SERVER, MAIL_LOGIN и MAIL_PASSWORD не пустые.

src/api/utils.py Outdated Show resolved Hide resolved
src/api/utils.py Outdated Show resolved Hide resolved
@gorskyolga gorskyolga merged commit 662fa21 into develop Sep 20, 2024
5 checks passed
@gorskyolga gorskyolga deleted the feature/script_to_invite_super_user branch September 20, 2024 07:52
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.

Добавить скрипт для приглашения первого супер-админа
3 participants