#30161: View decorators docs should mention how to decorate class based views
------------------------------------------------+------------------------
Reporter: Jaap Roes | Owner: nobody
Type: Cleanup/optimization | Status: new
Component: Documentation | Version: master
Severity: Normal | Keywords:
Triage Stage: Unreviewed | Has patch: 0
Needs documentation: 0 | Needs tests: 0
Patch needs improvement: 0 | Easy pickings: 0
UI/UX: 0 |
------------------------------------------------+------------------------
Currently the docs on
"[https://docs.djangoproject.com/en/2.1/topics/http/decorators/ View
decorators]" don't mention how to apply a decorator to a class based view.
There are instructions on how to do this in the
"[https://docs.djangoproject.com/en/2.1/topics/class-based-views/intro
/#decorating-class-based-views Decorating class-based views]" section of
the "Introduction to class-based views" part of the docs.
To me it seems reasonable to link to that section, or even include (part)
of it in the "View decorators" part of the docs.
Another minor note, the "Decorating class-based views" section has
instructions on how to apply the `login_required` decorator, but doesn't
mention the alternative `LoginRequiredMixin` which is only mentioned in
the "[https://docs.djangoproject.com/en/2.1/topics/auth/default/#the-
loginrequired-mixin Using the Django authentication system]" part of the
docs.
--
Ticket URL: <https://code.djangoproject.com/ticket/30161>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
--
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
To view this discussion on the web visit
https://groups.google.com/d/msgid/django-updates/048.a4db827b5ae6d6a14bc0178a83d4d237%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
