#25701: Add warning to an admin list_view if too many queries are being used
-------------------------------+------------------------------------
Reporter: jacinda | Owner: jacinda
Type: New feature | Status: assigned
Component: contrib.admin | Version: master
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------+------------------------------------
Changes (by timgraham):
* version: => master
* stage: Unreviewed => Accepted
Comment:
Using the console log handler seems reasonable to me (maybe at the INFO
level?). Aymeric, could you elaborate on why you want a `ModelAdmin`
option to disable it? I think it would be easy enough to ignore the
message if you don't care about it, but maybe there's some other reason
you had in mind? If we do add such an option, I wonder if it's better
suited to `ChangeList` (which we'd likely need to start documenting as a
public API).
Related the last point, Ola pointed out that the
`ModelAdmin.get_search_results()` method really belongs on `ChangeList`,
but we put it on `ModelAdmin` because the latter class isn't documented
and requires a bit more work to override (using
`ModelAdmin.get_changelist()`).
--
Ticket URL: <https://code.djangoproject.com/ticket/25701#comment:6>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
--
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit
https://groups.google.com/d/msgid/django-updates/065.db7b7cc7a469aeeba22161d09972d5f1%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
