Opened 8 years ago

Closed 8 years ago

#4222 closed (wontfix)

Make ChangeList overridable in ModelAdmin

Reported by: Øyvind Saltvik <oyvind@…> Owned by: adrian
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@…> 8 years ago.
proposed patch

Download all attachments as: .zip

Change History (4)

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

proposed patch

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

  • Needs documentation set
  • Needs tests unset
  • Patch needs improvement unset
  • Triage Stage changed from Unreviewed to Design decision needed

comment:2 Changed 8 years ago by adrian

  • Version changed from newforms branch to newforms-admin

comment:3 Changed 8 years ago by adrian

  • Resolution set to wontfix
  • Status changed from new to closed

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