Это потребует огромной работы по рефакторингу кода на этом уровне, учитывая тот факт, что все представления уже работают без сбоев. Но это должно уменьшить количество фактического кода и внедрить более актуальные и желательные практики при разработке проекта.
кто-нибудь работает над этим?
Привет @anweshknayak Не сейчас. Вы заинтересованы в работе над этим?
@ sebastian-code Да, я. Позвольте мне настроить этот проект. Спрошу по любому запросу
Хорошо, @anweshknayak , я постараюсь помочь.
@ sebastian-code Должен ли я делать разные PR для каждого приложения?
Привет @anweshknayak, я думаю, это лучший подход.
@ sebastian-code не могли бы вы подсказать, какой вид использовать? (https://ccbv.co.uk/) Думаю, мы можем использовать DetailView или ListView?
Начиная отсюда (https://github.com/vitorfs/bootcamp/blob/master/bootcamp/activities/views.py#L10)
Привет, @anweshknayak , это будет список объектов (уведомлений), поэтому правильным подходом было бы наследование от ListView, также не забудьте наследовать от собственного миксина LogginRequired в представление.
@ sebastian-code, вы хотите, чтобы каждое представление было переписано как cbv?
Что ж, не обязательно @tamirOK CBV в основном нацелены на снижение многословности и избежание случаев _DRY_, поэтому я полагаю, что в некоторых случаях имеет больше смысла иметь функциональное представление, и это требует применения логики, чтобы избежать ненужной работы. Я бы сказал, чем оставляю это на ваше усмотрение, хотите ли вы что-то дать проекту.
Любая помощь приветствуется.
Но не забудьте добавить тест, одна из целей - улучшить процент покрытия.
Закрытие этого вопроса, потому что оно слишком широкое и неконкретное, что делает его бесполезным. И потому что эта цель уже включена в №66, поэтому мы можем идти в ногу с ней.