Opened 9 years ago

Closed 9 years ago

#4222 closed (wontfix)

Make ChangeList overridable in ModelAdmin

Reported by: Øyvind Saltvik <oyvind@…> Owned by: Adrian Holovaty
Component: contrib.admin Version: newforms-admin
Severity: Keywords: override ChangeList
Cc: Triage Stage: Design decision needed
Has patch: yes Needs documentation: yes
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:

Description

Make ChangeList overridable in ModelAdmin to be able to set default fillterspecs etc.

Attachments (1)

changelist_override.diff (881 bytes) - added by Øyvind Saltvik <oyvind@…> 9 years ago.
proposed patch

Download all attachments as: .zip

Change History (4)

Changed 9 years ago by Øyvind Saltvik <oyvind@…>

Attachment: changelist_override.diff added

proposed patch

comment:1 Changed 9 years ago by Simon G. <dev@…>

Needs documentation: set
Needs tests: unset
Patch needs improvement: unset
Triage Stage: UnreviewedDesign decision needed

comment:2 Changed 9 years ago by Adrian Holovaty

Version: newforms branchnewforms-admin

comment:3 Changed 9 years ago by Adrian Holovaty

Resolution: wontfix
Status: newclosed

I think that, instead of this approach, it'd be better to "split up" the various methods/options that ChangeList provides, and make bona fide hooks on ModelAdmin. The goal should be to abolish the concept of the ChangeList object entirely, so the ModelAdmin provides all of those hooks.

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