subject:"\[Django\] #29999\: Include overriding admin site to tutorial"

Re: [Django] #29999: Include overriding admin site to tutorial

2018-12-03 Thread Django

#2: Include overriding admin site to tutorial
-+-
 Reporter:  Jan Musílek  |Owner:  nobody
 Type:   |   Status:  closed
  Cleanup/optimization   |
Component:  Documentation|  Version:  2.1
 Severity:  Normal   |   Resolution:  wontfix
 Keywords:   | Triage Stage:
 |  Unreviewed
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  1|UI/UX:  0
-+-
Changes (by Tim Graham):

 * status:  new => closed
 * resolution:   => wontfix


-- 
Ticket URL: 
Django 
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/067.cdbab54a9ad66301b55cf6cc827a1578%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #29999: Include overriding admin site to tutorial

2018-12-03 Thread Django

#2: Include overriding admin site to tutorial
-+-
 Reporter:  Jan Musílek  |Owner:  nobody
 Type:   |   Status:  new
  Cleanup/optimization   |
Component:  Documentation|  Version:  2.1
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:
 |  Unreviewed
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  1|UI/UX:  0
-+-

Comment (by Rodrigo):

 I agree with Tim, overriding the admin site seems out of the scope of an
 introductory tutorial to me. If mentioned, I think it should be with
 something like "In an actual project, if you need to heavily customize the
 admin, it can be entirely overriden with...", not as a way of changing the
 site header - which is what the site.site_header attr is for

-- 
Ticket URL: 
Django 
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/067.ffea425bd2f753dcd6d70e36c4e1ee81%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #29999: Include overriding admin site to tutorial

2018-12-03 Thread Django

Comment (by Jan Musílek):

Replying to [comment:1 Tim Graham]:
> I don't see overriding the admin site as a necessary requirement.
`site.site_header = '...'` still works fine.
Sure, it isn't necessary, I just see it as a new obvious way how to
customize admin site in broader context. Is it the recommended way? I
thought it might, which is why I proposed this change in the first place.
Would it be better if we changed **and** to **or**?

* We use this approach to teach you how to override templates. In an
actual project, you would probably
[https://docs.djangoproject.com/en/2.1/ref/contrib/admin/#overriding-
default-admin-site override the default admin site] **or** use

[https://docs.djangoproject.com/en/2.1/ref/contrib/admin/#django.contrib.admin.AdminSite.site_header
django.contrib.admin.AdminSite.site_header] attribute to more easily make
this particular customization.

--
Ticket URL:
Django
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/067.33757eb460ec905db242556fa9f3fb3a%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #29999: Include overriding admin site to tutorial

2018-11-30 Thread Django

#2: Include overriding admin site to tutorial
-+-
 Reporter:  Jan Musílek  |Owner:  nobody
 Type:   |   Status:  new
  Cleanup/optimization   |
Component:  Documentation|  Version:  2.1
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:
 |  Unreviewed
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  1|UI/UX:  0
-+-

Comment (by Tim Graham):

 I don't see overriding the admin site as a necessary requirement.
 `site.site_header = '...'` still works fine.

-- 
Ticket URL: 
Django 
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/067.cd80a7bc93242457ddd5f1fb993d0d32%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

[Django] #29999: Include overriding admin site to tutorial

2018-11-30 Thread Django

#2: Include overriding admin site to tutorial
+
Reporter: stinovlas | Owner: nobody
Type: Cleanup/optimization | Status: new
Component: Documentation |Version: 2.1
Severity: Normal| Keywords:
Triage Stage: Unreviewed| Has patch: 0
Needs documentation: 0 |Needs tests: 0
Patch needs improvement: 0 | Easy pickings: 1
UI/UX: 0 |
+
Django 2.1 introduced ability to
[https://docs.djangoproject.com/en/2.1/ref/contrib/admin/#overriding-
default-admin-site override default admin site], which is a great feature.
In the [https://docs.djangoproject.com/en/2.1/intro/tutorial07/#customize-
the-admin-look-and-feel tutorial], change of the heading is achieved by
overriding templates. I'm aware that it's done so because of teaching new
users how to override templates (as stated in the tutorial itself), but I
think that ability to override default admin should be mentioned in this
place as well.

In the spirit of ''"There should be one – and preferably only one –
obvious way to do it."'' I think that overriding the default admin site is
the obvious way to customize django admin site from now on and therefore,
it should be mentioned in the tutorial.

**Currently, the documentation states (this paragraph has not been changed
at least from Django 1.9):**

* We use this approach to teach you how to override templates. In an
actual project, you would probably use the

**I propose to change this paragraph thus:**

--
Ticket URL:
Django
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/052.ade7f7c2d476a66453945f215b8a6fc8%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #29999: Include overriding admin site to tutorial

Re: [Django] #29999: Include overriding admin site to tutorial

Re: [Django] #29999: Include overriding admin site to tutorial

Re: [Django] #29999: Include overriding admin site to tutorial

[Django] #29999: Include overriding admin site to tutorial

5 matches

Site Navigation

Mail list logo

Footer information