Opened 16 years ago

Last modified 15 years ago

#6286 closed

tutorial part 3 - Design your urls misses admin views — at Version 10

Reported by: Bashar Al-Abdulhadi Owned by: John DeRosa
Component: Documentation Version: dev
Severity: Keywords: tutorial
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description (last modified by Ramiro Morales)

In Chapter 3 Design your urls section, the urlpatterns assignment seems to miss the admin url patter from the assignment:

urlpatterns = patterns('',
    (r'^polls/$', 'mysite.polls.views.index'),
    (r'^polls/(?P<poll_id>\d+)/$', 'mysite.polls.views.detail'),
    (r'^polls/(?P<poll_id>\d+)/results/$', 'mysite.polls.views.results'),
    (r'^polls/(?P<poll_id>\d+)/vote/$', 'mysite.polls.views.vote'),
)

Perhaps it's because the next step is to move that section to the polls directory, but it may lead people to overwrite the admin section unaware.

Change History (11)

comment:1 by arien, 16 years ago

Resolution: duplicate
Status: newclosed

Duplicate of #6190.

comment:2 by arien, 16 years ago

Needs documentation: set
Resolution: duplicate
Status: closedreopened
Triage Stage: UnreviewedAccepted

Ugh... Sorry for the close. This isn't a duplicate of #6190, and moreover #6190 looks bogus. Reopening.

comment:3 by arien, 16 years ago

The admin views are not included in the urls.py on purpose. Including them would result in ViewDoesNotExist exceptions when trying to convert URLs to views, because the polls views haven't been written yet. This means the admin is not available until all the views are written (at the end of the section before Use generic views: Less code is better in the fourth part of the tutorial).

Adding a sentence that tells the reader to put the admin views back in when all the views are written (and maybe explaining why they were temporarily taken out?) would perhaps prevent confusion.

comment:4 by arien, 16 years ago

Keywords: tutorial added; urls.py removed
Summary: Chapter 3 - Design your urls misses Admin sectiontutorial part 3 - Design your urls misses admin views

comment:5 by Tyler Mitchell <spatialguru@…>, 16 years ago

Came across this too - as a new user reading the tutorial I got stumped. Wasn't sure whether to copy verbatim or leave it the admin pattern in there.

comment:6 by John DeRosa, 16 years ago

Owner: changed from nobody to John DeRosa
Status: reopenednew

by John DeRosa, 16 years ago

Attachment: 6286.patch added

comment:7 by John DeRosa, 16 years ago

Has patch: set

Patch file attached.

Note, svn thinks every line changed due to bug #6545.

comment:8 by John DeRosa, 16 years ago

Needs documentation: unset

Oops, I forgot to clear "Needs documentation."

comment:9 by John DeRosa, 16 years ago

Status: newassigned

comment:10 by Ramiro Morales, 16 years ago

Description: modified (diff)
Note: See TracTickets for help on using tickets.
Back to Top