#21685 closed Cleanup/optimization (fixed)
admin_doc/model_index.html should use application names
| Reported by: | Aymeric Augustin | Owned by: | mrshu |
|---|---|---|---|
| Component: | contrib.admindocs | Version: | dev |
| Severity: | Normal | Keywords: | app-loading |
| Cc: | Triage Stage: | Accepted | |
| Has patch: | no | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | yes | UI/UX: | no |
Description
Currently, the template uses app_label|capfirst to display the application name.
Since the app loading refactor, app_config.verbose_name provides a customizable application name.
Attachments (1)
Change History (8)
comment:1 by , 11 years ago
| Triage Stage: | Unreviewed → Accepted |
|---|
comment:2 by , 11 years ago
| Easy pickings: | set |
|---|
comment:3 by , 11 years ago
comment:4 by , 11 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → assigned |
I'd like to try to create a patch for this but I could not find app_config being used anywhere.
Could you please elaborate more on how to you imagine this patch to look like?
Thanks!
comment:5 by , 11 years ago
Here's the commit that did that change in django.contrib.admin:
https://github.com/django/django/commit/ba60fcbcf7645afd82f3135ecb56562f8a9c9824
This ticket is essentially about doing the same thing in django.contrib.admindocs.
The appropriate app_config object must be added to the template context.
comment:6 by , 11 years ago
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
Note:
See TracTickets
for help on using tickets.
Anyone wants to provide a patch for this? I don't expect it to be particularly hard.
It's low priority for me because it's peripheral to app-loading.
Tests are appreciated but not absolutely required given admindocs' relatively low test coverage.