AS
Нужно структуру менять, чтобы полностью разделить фронт от бека, и разнести их в разные сервисы
👍
Size: a a a
2019 February 07
2019 February 08
А
Добавил докер, пока wip, нужно оформить readme
мне кажется, слишком много изменений в settings.py
нужно сохранять хотя бы частичную совместимость, если нет больших плюсов у нового формата
нужно сохранять хотя бы частичную совместимость, если нет больших плюсов у нового формата
M
Согласен, предлагаю делать small changes в проекте. Завтра тоже выкачу PR с докером, но я отталкивался от текущей конфигурации, не трогая основной код. Давайте сравним и придем к консенсусу.
Ещё завтра будет CI через Travis, но я в сомнениях, что конкретно тестировать. Пока сделал лишь встроенный тест django и сценарий по запуску приложения и проверке, что оно отвечает 200 на http-get. Что ещё надо проверять?
Ещё завтра будет CI через Travis, но я в сомнениях, что конкретно тестировать. Пока сделал лишь встроенный тест django и сценарий по запуску приложения и проверке, что оно отвечает 200 на http-get. Что ещё надо проверять?
DI
мне кажется, слишком много изменений в settings.py
нужно сохранять хотя бы частичную совместимость, если нет больших плюсов у нового формата
нужно сохранять хотя бы частичную совместимость, если нет больших плюсов у нового формата
Я бд не трогал, все работает. А конфигурирование через один файл проще
DI
Согласен, предлагаю делать small changes в проекте. Завтра тоже выкачу PR с докером, но я отталкивался от текущей конфигурации, не трогая основной код. Давайте сравним и придем к консенсусу.
Ещё завтра будет CI через Travis, но я в сомнениях, что конкретно тестировать. Пока сделал лишь встроенный тест django и сценарий по запуску приложения и проверке, что оно отвечает 200 на http-get. Что ещё надо проверять?
Ещё завтра будет CI через Travis, но я в сомнениях, что конкретно тестировать. Пока сделал лишь встроенный тест django и сценарий по запуску приложения и проверке, что оно отвечает 200 на http-get. Что ещё надо проверять?
-
на первоначальном этапе сделать хорошо с минимальными изменениями невозможно
на первоначальном этапе сделать хорошо с минимальными изменениями невозможно
DI
Почему сейчас нужно было это делать?.. Сейчас нет как таковой архитектуры проекта, сделано быстро - на коленке, поэтому исправлять лучше сразу.
DI
Структура проекта плохая, всё собрано в кучу, фронт, бек. Из дрф используются только сериализаторы, фильтрации как таковой почти нет, как сейчас это жестоко.
DI
Парсер вообще странный, один только
eval для преобразования json в dict чего стоитMA
Парсер вообще странный, один только
eval для преобразования json в dict чего стоитAst нужен
DI
json.loads нужен
MA
А, я думал там строка просто. Да, loads
DI
там строка, которая приходит с сервера
DI
что-то из этого выделил в issues
DI
Я пришёл в проект помочь с кодом, по факту вы ограничиваете изменения по кол-ву строк...
DI
По меньшей мере это странно 😉
M
-
на первоначальном этапе сделать хорошо с минимальными изменениями невозможно
на первоначальном этапе сделать хорошо с минимальными изменениями невозможно
Я к тому, что делать можно постепенно, даже если нет архитектуры. Ну и разделять большие таски на микротаски, потому что контейнеры это она задача, архитектура - другая... В одном PR никто не захочет рассматривать сразу несколько задач, это я понимаю )
Понятно, что хочется всего и сразу.
Понятно, что хочется всего и сразу.
DI
Там одна задача зафигачить всё в докер
DI
А держать в докере текстовые файлы, читаемые "вручную" это странно, притом что есть нормальный вариант с хранением настроек в environment
DI
Которые автоматом подхватываются приложением