alex/django-todo
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Feat 6 #18

Merged
Derinhelm merged 3 commits from feat_6 into develop 2021-04-14 13:37:40 +03:00
Conversation 3 Commits 3 Files Changed 3
+28 -1
Derinhelm commented 2021-04-14 00:21:46 +03:00 (Migrated from github.com)
Copy Link
Copy Source
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
Copy Link
Copy Source

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

Мне кажется, пока можно не мержить, т.к. задача неблокирующая.
backend/backend/api.py
AlekseyLobanov (Migrated from github.com) commented 2021-04-14 00:26:37 +03:00
Copy Link
Copy Source

Вроде этот 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
Copy Link
Copy Source

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

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
Copy Link
Copy Source

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

Тут лучше зафиксировать версию, которую ты используешь
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
Copy Link
Copy Source

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

Пожалуйста, зафиксируй версию через == иначе выйдет новая и всё сломает :(
AlekseyLobanov (Migrated from github.com) approved these changes 2021-04-14 13:19:42 +03:00
AlekseyLobanov (Migrated from github.com) left a comment
Copy Link
Copy Source

Стало лучше
LGTM

Стало лучше LGTM
Sign in to join this conversation.
Reviewers
No Reviewers
AlekseyLobanov
Labels
Clear labels
bug
Something isn't working
 documentation
Improvements or additions to documentation
 duplicate
This issue or pull request already exists
 enhancement
New feature or request
 good first issue
Good for newcomers
 help wanted
Extra attention is needed
 invalid
This doesn't seem right
 question
Further information is requested
 wontfix
This will not be worked on
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
alex (Aleksey Lobanov)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: alex/django-todo#18