Django

Code

Ticket #8498 (closed: fixed)

Opened 2 years ago

Last modified 2 years ago

Make template for new django.contrib.admin.sites.AdminSite.app_index() configurable

Reported by: jezdez Assigned to: jezdez
Milestone: 1.0 Component: django.contrib.admin
Version: SVN Keywords:
Cc: Triage Stage: Unreviewed
Has patch: 1 Needs documentation: 0
Needs tests: 0 Patch needs improvement: 0

Description

I very much welcome r8474 but think the template used in the new app_index method needs to be configurable, just like the display_login_form() and index() method.

Currently only 'admin/app_index.html' is allowed and limits the usefulness of inheriting of the AdminSite class. I stumbled over this while working on the new nfa-aware mobileadmin.

I suggest adding a class attribute (e.g.: app_index_template = None).

Attachments

ticket8498-r8486.1.diff (1.0 kB) - added by jezdez on 08/23/08 10:05:53.
Small patch to add support for app_index_template

Change History

08/23/08 10:05:53 changed by jezdez

  • attachment ticket8498-r8486.1.diff added.

Small patch to add support for app_index_template

08/23/08 10:55:54 changed by jezdez

  • needs_better_patch changed.
  • has_patch set to 1.
  • needs_tests changed.
  • needs_docs changed.

08/23/08 10:56:03 changed by jezdez

  • owner changed from nobody to jezdez.
  • status changed from new to assigned.

08/23/08 11:27:12 changed by brosner

  • status changed from assigned to closed.
  • resolution set to fixed.

(In [8487]) Fixed #8498 -- Add the ability to customize the template for app_index similar to the other views on the AdminSite? class. Thanks Jannis Leidel for the patch.

Add/Change #8498 (Make template for new django.contrib.admin.sites.AdminSite.app_index() configurable)




Change Properties
Action