[Django] #28886: Two different urls set are suggested for start with the authentication subsystem

13 views
Skip to first unread message

Django

unread,
Dec 5, 2017, 4:28:15 AM12/5/17
to django-...@googlegroups.com
#28886: Two different urls set are suggested for start with the authentication
subsystem
-----------------------------------------+------------------------
Reporter: David Mugnai | Owner: nobody
Type: Uncategorized | Status: new
Component: Documentation | Version: 1.11
Severity: Normal | Keywords:
Triage Stage: Unreviewed | Has patch: 0
Needs documentation: 0 | Needs tests: 0
Patch needs improvement: 0 | Easy pickings: 0
UI/UX: 0 |
-----------------------------------------+------------------------
In the "The login_required decorator" paragraph of the "Using the Django
authentication system" guide is stated that the default url for the login
view is "/accounts/login", but in the "Authentication Views" section all
the examples don't use the "accounts/" part.

I think the examples should be changed to start with this code:
{{{
urlpatterns = [
path('^accounts$', include('django.contrib.auth.urls')),
]
}}}

--
Ticket URL: <https://code.djangoproject.com/ticket/28886>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

Django

unread,
Dec 5, 2017, 3:05:35 PM12/5/17
to django-...@googlegroups.com
#28886: Example included auth urls don't include accounts/ prefix assumed by much
of the other documentation
--------------------------------------+------------------------------------

Reporter: David Mugnai | Owner: nobody
Type: Cleanup/optimization | Status: new
Component: Documentation | Version: 1.11
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 Tim Graham):

* type: Uncategorized => Cleanup/optimization
* stage: Unreviewed => Accepted


Comment:

[https://github.com/django/django/pull/9431 PR]

--
Ticket URL: <https://code.djangoproject.com/ticket/28886#comment:1>

Django

unread,
Dec 6, 2017, 9:23:57 AM12/6/17
to django-...@googlegroups.com
#28886: Example included auth urls don't include accounts/ prefix assumed by much
of the other documentation
--------------------------------------+------------------------------------
Reporter: David Mugnai | Owner: nobody
Type: Cleanup/optimization | Status: closed
Component: Documentation | Version: 1.11
Severity: Normal | Resolution: fixed
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 GitHub <noreply@…>):

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


Comment:

In [changeset:"4114b441ee63e5e9196f20beee5cb071d1a21b3b" 4114b441]:
{{{
#!CommitTicketReference repository=""
revision="4114b441ee63e5e9196f20beee5cb071d1a21b3b"
Fixed #28886 -- Updated prefix for example django.contrib.auth.urls URLs.
}}}

--
Ticket URL: <https://code.djangoproject.com/ticket/28886#comment:2>

Django

unread,
Dec 6, 2017, 9:24:22 AM12/6/17
to django-...@googlegroups.com
#28886: Example included auth urls don't include accounts/ prefix assumed by much
of the other documentation
--------------------------------------+------------------------------------
Reporter: David Mugnai | Owner: nobody
Type: Cleanup/optimization | Status: closed
Component: Documentation | Version: 1.11

Severity: Normal | Resolution: fixed
Keywords: | Triage Stage: Accepted
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
--------------------------------------+------------------------------------

Comment (by Tim Graham <timograham@…>):

In [changeset:"e30674a1b777c56a28734b569063ee4395751515" e30674a]:
{{{
#!CommitTicketReference repository=""
revision="e30674a1b777c56a28734b569063ee4395751515"
[2.0.x] Fixed #28886 -- Updated prefix for example
django.contrib.auth.urls URLs.

Backport of 4114b441ee63e5e9196f20beee5cb071d1a21b3b from master
}}}

--
Ticket URL: <https://code.djangoproject.com/ticket/28886#comment:3>

Reply all
Reply to author
Forward
0 new messages