Bootcamp: Mise en œuvre du CBV

Créé le 30 mai 2016  ·  11Commentaires  ·  Source: vitorfs/bootcamp

Cela nécessiterait une énorme quantité de travail pour refactoriser le code à ce niveau, étant donné que toutes les vues fonctionnent déjà sans problème. Mais cela devrait réduire la quantité de code réel et mettre en œuvre des pratiques plus réelles et souhaitables sur le développement du projet.

enhancement help wanted

Tous les 11 commentaires

quelqu'un travaille dessus ?

Salut @anweshknayak Pas maintenant. Êtes-vous intéressé à travailler sur cela?

@sebastian-code Oui je le suis. Permettez-moi de configurer ce projet. je vous demanderai sur toute question

Ok @anweshknayak , je ferais de mon mieux pour aider.

@sebastian-code dois-je faire un PR différent pour chaque application ?

Salut @anweshknayak, je pense que c'est la meilleure approche.

@sebastian-code pouvez-vous donner un indice sur le type de vue à utiliser ? (https://ccbv.co.uk/) Je pense que nous pouvons utiliser DetailView ou ListView ?
À partir d'ici (https://github.com/vitorfs/bootcamp/blob/master/bootcamp/activities/views.py#L10)

Salut @anweshknayak ce serait une liste d'objets (notifications) donc la bonne approche serait d'hériter de ListView, aussi, n'oubliez pas d'hériter du mixin natif LogginRequired dans la vue.

@sebastian-code, voulez-vous que chaque vue soit réécrite en cbv ?

Bon, pas forcément @tamirOK Les CBV visent principalement à réduire la verbosité et à éviter les cas de _DRY_, donc j'imagine que dans certains cas il est plus logique d'avoir une vue fonctionnelle, et cela nécessite d'appliquer une logique pour éviter un travail inutile. Je dirais que je vous laisse le soin de savoir si vous voulez donner quelque chose au projet.

Toute aide est appréciée.

Mais n'oubliez pas d'ajouter un test, l'un des objectifs est d'améliorer le pourcentage de couverture.

Fermer ce problème car il est trop large et peu spécifique, ce qui le rend inutile. Et parce que c'est un objectif déjà inclus dans #66 afin que nous puissions le suivre là-bas.

Cette page vous a été utile?
0 / 5 - 0 notes

Questions connexes

Shekharnunia picture Shekharnunia  ·  6Commentaires

mwanjajoel picture mwanjajoel  ·  5Commentaires

phamminhtris picture phamminhtris  ·  12Commentaires

Allan-Nava picture Allan-Nava  ·  9Commentaires

sebastian-code picture sebastian-code  ·  11Commentaires