New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Implementing CBV #50
Comments
anyone working on this ? |
Hi @anweshknayak Not right now. Are you interested into working on this? |
@sebastian-code Yes I am. Let me setup this project. I will ask you upon any query |
Ok @anweshknayak , I would try my best to help. |
@sebastian-code shall I make a different PR for each app ? |
Hi @anweshknayak I would think is the best approach. |
@sebastian-code can you give some hint on what kind of view to use ? ( https://ccbv.co.uk/) I think we can use DetailView or ListView ? |
Hi @anweshknayak that would be a list of objects (notifications) so the right approach would be to inherit from ListView, also, don't forget to inherit from the native LogginRequired mixin into the view. |
@sebastian-code, do you want every view to be rewritten as cbv? |
Well, not necessarily @tamirOK CBV are mainly aimed to reduce verbosity and to avoid cases of DRY, so I imagine than in some cases it makes more sense to have a functional view, and that requires to apply logic to avoid unnecessary work. I would say than I leave it to you were do you want to give something to the project. Any help is appreciated. But don't forget to add test, one of the goals is to improve the coverage percentage. |
Closing this issue because is to wide and unspecific, which makes it unhelpful. And because is a goal already included in #66 so we can keep up with it there. |
This would require a huge amount of work to refactor the code at this level, given the fact than all the views works already smoothly. But this should reduce the amount of actual code and would implement more actual and desirable practices on the project development.
The text was updated successfully, but these errors were encountered: