оптимизировать sorl-thumbnale, чтобы был только один запрос, а не на каждую картинку отдельно. В catalog_list DjDT пишет: 2 similar queries
В select_related users_list доставать только image, а не весь user.profile.related.name
Сделать менеджер модели User (в семинаре урока "Работа с моделью пользователя" кое-что показывали)
Рекомендации ревьюера:
Все лишние пустые файлы убраны, все старые свои и обучающие комменты из их начала убраны (вы уже без них знаете и о нужных файлах, и что в них должно быть)
Стоит вспомнить про оптимизацию количества миграций и про актуальность фикстур и демо-БД (там логин-пароль все произвольно выбирали, хотя я тогда рекомендовал классическое и оптимальное admin-admin, к чему можете вернуться). Про хорошее размещение файлов (на встрече упоминал, например, про файлы зависимостей с отдельной папке requirements, чтобы не мозолили глаза в корне, ну а там prod.txt, dev.txt, test.txt).
Дополнительное от ревьюера:
Тесты чтобы были все аккуратно, в своих файлах, где надо — с сетапом (каталог, тег, возможно клиент, что там ещё) и тирдауном. Где применимо — фикстуры задействовать (опционально). Сейчас свежим взглядом может захотеться что-то уточнить или унифицировать.
Ещё поиграйтесь на разных страницах с изменением (уменьшением) ширины окна браузера. Возможно, вёрстка начнёт немного неожиданно скакать (например, карточки или промежутки рядом с ними), и это косяк не бутстрапа, а вёрстки.
Дальше визуально посмотрите на свой код, чтобы легко читался. Возможно, где-то сами увидите, что можно написать чуть проще или легче для восприятия, уточнить нейминг и т.п.
убрать относительные импорты (особенно с
..
)оптимизировать sorl-thumbnale, чтобы был только один запрос, а не на каждую картинку отдельно. В catalog_list DjDT пишет: 2 similar queries
В select_related users_list доставать только image, а не весь user.profile.related.name
Сделать менеджер модели User (в семинаре урока "Работа с моделью пользователя" кое-что показывали)
Рекомендации ревьюера:
Все лишние пустые файлы убраны, все старые свои и обучающие комменты из их начала убраны (вы уже без них знаете и о нужных файлах, и что в них должно быть)
Стоит вспомнить про оптимизацию количества миграций и про актуальность фикстур и демо-БД (там логин-пароль все произвольно выбирали, хотя я тогда рекомендовал классическое и оптимальное admin-admin, к чему можете вернуться). Про хорошее размещение файлов (на встрече упоминал, например, про файлы зависимостей с отдельной папке requirements, чтобы не мозолили глаза в корне, ну а там prod.txt, dev.txt, test.txt).
Дополнительное от ревьюера: Тесты чтобы были все аккуратно, в своих файлах, где надо — с сетапом (каталог, тег, возможно клиент, что там ещё) и тирдауном. Где применимо — фикстуры задействовать (опционально). Сейчас свежим взглядом может захотеться что-то уточнить или унифицировать.
Ещё поиграйтесь на разных страницах с изменением (уменьшением) ширины окна браузера. Возможно, вёрстка начнёт немного неожиданно скакать (например, карточки или промежутки рядом с ними), и это косяк не бутстрапа, а вёрстки.
Дальше визуально посмотрите на свой код, чтобы легко читался. Возможно, где-то сами увидите, что можно написать чуть проще или легче для восприятия, уточнить нейминг и т.п.