Opened 10 years ago
Closed 10 years ago
#23573 closed Cleanup/optimization (duplicate)
Should tutorial 2 demonstrate AdminSite.site_header instead of overriding templates?
Reported by: | hernandocounty | Owned by: | nobody |
---|---|---|---|
Component: | Documentation | Version: | 1.7 |
Severity: | Normal | Keywords: | |
Cc: | hernandocounty | Triage Stage: | Accepted |
Has patch: | no | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description (last modified by )
Tutorial 2 shows how to change the admin's header (see also #21378).
However, in Django 1.7, there's a better way to achieve this particular task: AdminSite.site_header.
If that example is intended to demonstrate how to override a template, then it should tackle another use case for which overriding templates is the simplest choice.
If it is intended to show how to customize the admin's header, then it should describe the simplest solution.
Change History (5)
comment:1 by , 10 years ago
Cc: | added |
---|---|
Needs documentation: | set |
comment:2 by , 10 years ago
Resolution: | → invalid |
---|---|
Status: | new → closed |
With such language, you are not going to go very far. If you really want to make a constructive contribution to the Django project, which is run through volunteers, please reopen a new ticket, change your language and describe politely what you are suggesting to improve in the Django documentation. Or just use another framework with better documentation.
comment:3 by , 10 years ago
Description: | modified (diff) |
---|
comment:4 by , 10 years ago
Component: | Template system → Documentation |
---|---|
Description: | modified (diff) |
Keywords: | TEMPLATES_DIR removed |
Needs documentation: | unset |
Resolution: | invalid |
Status: | closed → new |
Summary: | Templates override with Admin Site TEMPLATE_DIRS wrong way → Should tutorial 2 demonstrate AdminSite.site_header instead of overriding templates? |
Triage Stage: | Unreviewed → Accepted |
I've rewritten the ticket because the state of the reporter's ass is only marginally relevant to the Django project ;-) However, the idea is valid.
comment:5 by , 10 years ago
Resolution: | → duplicate |
---|---|
Status: | new → closed |
Mmm, in fact the discussion reached the same point in #23575 (unless I missed something).
https://docs.djangoproject.com/en/1.7/intro/tutorial02/
this page has bad info
this page is how I finally changed the text on the admin interface.
http://stackoverflow.com/questions/4938491/django-admin-change-header-django-administration-text