Bootcamp: Implementing CBV

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 ?
Starting from here (https://github.com/vitorfs/bootcamp/blob/master/bootcamp/activities/views.py#L10)

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.