#28777 closed Cleanup/optimization (fixed)
Clarify the docs for admin's @register's arguments
| Reported by: | Matt Swan | Owned by: | nobody |
|---|---|---|---|
| Component: | Documentation | Version: | 1.11 |
| Severity: | Normal | Keywords: | register decorator |
| Cc: | Triage Stage: | Ready for checkin | |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | yes | UI/UX: | no |
Description (last modified by )
The description text for the register decorator has a complex sentence that is badly worded and not proper english, meaning that the functions usage is not properly described.
This page:
https://docs.djangoproject.com/en/1.11/ref/contrib/admin/
Please reword this sentence so it make sense:
It is given one or more model classes to register with the ModelAdmin and an optional keyword argument site if you are not using the default AdminSite:
Suggested changes - but I don't understand the sentence so the suggestion is just a guess:
"It is given IF/FOR(?) one or more model classes to register with the ModelAdmin and an optional keyword argument site FULL STOP. If you are not using the default AdminSite:"
Change History (5)
comment:1 by , 8 years ago
| Description: | modified (diff) |
|---|
comment:2 by , 8 years ago
| Has patch: | set |
|---|---|
| Summary: | The register decorator description is badly worded - needs clarification → Clarify the docs for admin's @register's arguments |
| Triage Stage: | Unreviewed → Accepted |
comment:3 by , 8 years ago
| Triage Stage: | Accepted → Ready for checkin |
|---|
Does this PR clarify?