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.
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.