Bootcamp: Внедрение CBV

Созданный на 30 мая 2016  ·  11Комментарии  ·  Источник: vitorfs/bootcamp

Это потребует огромной работы по рефакторингу кода на этом уровне, учитывая тот факт, что все представления уже работают без сбоев. Но это должно уменьшить количество фактического кода и внедрить более актуальные и желательные практики при разработке проекта.

enhancement help wanted

Все 11 Комментарий

кто-нибудь работает над этим?

Привет @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, поэтому мы можем идти в ногу с ней.

Была ли эта страница полезной?
0 / 5 - 0 рейтинги

Смежные вопросы

phamminhtris picture phamminhtris  ·  12Комментарии

Shekharnunia picture Shekharnunia  ·  10Комментарии

norouzzadeh picture norouzzadeh  ·  3Комментарии

sebastian-code picture sebastian-code  ·  11Комментарии

mwanjajoel picture mwanjajoel  ·  5Комментарии