#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 , 10 years ago
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
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.