Vasar007
commented
6 years ago @levozavr, не забудь потом отредактировать комментарии. Пример хороших комментариев ты можешь найти в репозиториях judyst-link-analysis и tinkoff-mlalgorithms.
Closed levkovalenko closed 6 years ago
Vasar007
commented
6 years ago @levozavr, не забудь потом отредактировать комментарии. Пример хороших комментариев ты можешь найти в репозиториях judyst-link-analysis и tinkoff-mlalgorithms.
danila19991
commented
6 years ago @Vasar007, как обработать случай, когда в модуль связанный с авторизацией посылает запрос авторизованный пользователь.
danila19991
commented
6 years ago @levozavr, Добавь описание скачивания и установки в readme.md.
levkovalenko
commented
6 years ago @danila19991 я не уверен что это приоритетная задача
Pull Request
ВАЖНО: Пожалуйста, не создавайте Pull Request, если не созданы issues, которые Ваш PR решает.
Описание
Для внесения изменений в основные ветки нужно получить разрешения от тимлидов команд или соответствующих ревьюверов Ваших репозиториев.
Пожалуйста, предоставьте достаточно информации о ваших изменениях для тех, кто будет ревьювить Ваш PR: добавлен следующий функционал:
Так же из-за поспешных решений был создан функционал не входящий в планы на этот PR поэтому просьба не смотреть SearchViewSet и все что с ним связанно, покрытие его тестами и рефакторинг будут проведены в другом PR.
Тестовый план
Покажите, что Ваш код действительно безопасный. Обязательно проверьте, чтобы изменения проходили CI и не ломали имеющийся код.
Функционал связанный с сохранением пользователей в базу данных и работой представления регистрации и авторизации был покрыт unit и системными тестами.
Закрытые issues
close #10 close #16 close #91 close #30 close #23 close #26 close #24 close #83 close #19 close #111