#30161 closed Cleanup/optimization (fixed)
View decorators docs should mention how to decorate class based views
| Reported by: | Jaap Roes | Owned by: | Alexandru Balan |
|---|---|---|---|
| Component: | Documentation | Version: | dev |
| Severity: | Normal | Keywords: | |
| Cc: | Triage Stage: | Accepted | |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
Currently the docs on "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 "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 "Using the Django authentication system" part of the docs.
Change History (5)
comment:1 by , 5 years ago
| Triage Stage: | Unreviewed → Accepted |
|---|
comment:2 by , 5 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → assigned |
comment:3 by , 5 years ago
| Has patch: | set |
|---|
Note:
See TracTickets
for help on using tickets.
PR