Feat 6 #18

2021-04-14T00:21:46+03:00

Derinhelm commented

2021-04-14 00:21:46 +03:00

(Migrated from github.com)

No description provided.

AlekseyLobanov (Migrated from github.com) requested changes 2021-04-14 00:27:14 +03:00

AlekseyLobanov (Migrated from github.com) left a comment

Мне кажется, пока можно не мержить, т.к. задача неблокирующая.

backend/backend/api.py

AlekseyLobanov (Migrated from github.com) commented

2021-04-14 00:26:37 +03:00

Вроде этот PR независим от других задач, поэтому сначала лучше доделать поддержку JWT
Вроде как это решение должно работать: https://github.com/axnsan12/drf-yasg/issues/46#issuecomment-357237872

Вроде этот PR независим от других задач, поэтому сначала лучше доделать поддержку JWT Вроде как это решение должно работать: https://github.com/axnsan12/drf-yasg/issues/46#issuecomment-357237872

backend/backend/settings.py

AlekseyLobanov (Migrated from github.com) commented

2021-04-14 00:25:23 +03:00

Я бы не ставил это без нужды. Или хотя бы

if DEBUG: 
    ALLOWED_HOSTS = ["0.0.0.0"]
else:
    ALLOWED_HOSTS = []

Я бы не ставил это без нужды. Или хотя бы ``` if DEBUG: ALLOWED_HOSTS = ["0.0.0.0"] else: ALLOWED_HOSTS = [] ```

backend/requirements.txt

AlekseyLobanov (Migrated from github.com) commented

2021-04-14 00:24:12 +03:00

Тут лучше зафиксировать версию, которую ты используешь

AlekseyLobanov (Migrated from github.com) reviewed 2021-04-14 13:18:50 +03:00

backend/requirements.txt

AlekseyLobanov (Migrated from github.com) commented

2021-04-14 13:18:50 +03:00

Пожалуйста, зафиксируй версию через == иначе выйдет новая и всё сломает :(

AlekseyLobanov (Migrated from github.com) approved these changes 2021-04-14 13:19:42 +03:00

AlekseyLobanov (Migrated from github.com) left a comment

Стало лучше
LGTM

Стало лучше LGTM

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: alex/django-todo#18