Opened 11 years ago

Closed 10 years ago

Last modified 10 years ago

#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)

admindocs.tar.gz (99.8 KB ) - added by alvaro@… 11 years ago.
path with some solutions.

Download all attachments as: .zip

Change History (8)

comment:1 by Aymeric Augustin, 11 years ago

Triage Stage: UnreviewedAccepted

comment:2 by Aymeric Augustin, 11 years ago

Easy pickings: set

comment:3 by Aymeric Augustin, 11 years ago

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.

comment:4 by mrshu, 11 years ago

Owner: changed from nobody to mrshu
Status: newassigned

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 Aymeric Augustin, 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.

by alvaro@…, 11 years ago

Attachment: admindocs.tar.gz added

path with some solutions.

comment:6 by Claude Paroz <claude@…>, 10 years ago

Resolution: fixed
Status: assignedclosed

In 4cf82e5bcf87e0d5419d941c2ef7ec8855801319:

Fixed #21685 -- Displayed app verbose name in admindocs model index

comment:7 by Claude Paroz <claude@…>, 10 years ago

In 9d0ebceb324ebb13b6d859153f95b2ef0c7c326b:

[1.7.x] Fixed #21685 -- Displayed app verbose name in admindocs model index

Backport of 4cf82e5bcf from master.

Note: See TracTickets for help on using tickets.
Back to Top