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

Финальное ревью #75

Merged
merged 25 commits into from
Jan 24, 2022
Merged

Финальное ревью #75

merged 25 commits into from
Jan 24, 2022

Conversation

shoom1337
Copy link
Contributor

No description provided.

@ivan-infro
Copy link

docker-compose.yaml в секции db перепутан DB_USER и DB_PASS

@ivan-infro
Copy link

Переменные DB_PASS и DB_PASSWORD (в db и app) приводят к путанице

@ivan-infro
Copy link

Доделать окончание игры

@ivan-infro
Copy link

Обратите внимание, что в .env файлах комментарии лучше оставлять на отдельной строке, иначе они воспринимаются как часть значения переменной

@ivan-infro
Copy link

Тема dark отсутствует в базе (запускал локально), при переключении возникает ошибка 500

@ivan-infro
Copy link

Обновите тесты

@Alkamenos
Copy link
Member

Исправил сломанные тесты

Signed-off-by: Leonid Artemev <alkamenos@gmail.com>
Signed-off-by: Leonid Artemev <alkamenos@gmail.com>
Signed-off-by: Leonid Artemev <alkamenos@gmail.com>
Signed-off-by: Leonid Artemev <alkamenos@gmail.com>
Signed-off-by: Leonid Artemev <alkamenos@gmail.com>
@Alkamenos Alkamenos merged commit dcd7c0d into main Jan 24, 2022
@Alkamenos Alkamenos deleted the final-review branch January 24, 2022 16:44
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