Opened 13 years ago
Closed 12 years ago
#6286 closed (fixed)
tutorial part 3 - Design your urls misses admin views
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 )
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.
Attachments (2)
Change History (14)
comment:1 Changed 13 years ago by
Resolution: | → duplicate |
---|---|
Status: | new → closed |
comment:2 Changed 13 years ago by
Needs documentation: | set |
---|---|
Resolution: | duplicate |
Status: | closed → reopened |
Triage Stage: | Unreviewed → Accepted |
comment:3 Changed 13 years ago by
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 Changed 13 years ago by
Keywords: | tutorial added; urls.py removed |
---|---|
Summary: | Chapter 3 - Design your urls misses Admin section → tutorial part 3 - Design your urls misses admin views |
comment:5 Changed 13 years ago by
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 Changed 13 years ago by
Owner: | changed from nobody to John DeRosa |
---|---|
Status: | reopened → new |
Changed 13 years ago by
Attachment: | 6286.patch added |
---|
comment:7 Changed 13 years ago by
Has patch: | set |
---|
Patch file attached.
Note, svn thinks every line changed due to bug #6545.
comment:8 Changed 13 years ago by
Needs documentation: | unset |
---|
Oops, I forgot to clear "Needs documentation."
comment:9 Changed 13 years ago by
Status: | new → assigned |
---|
comment:10 Changed 13 years ago by
Description: | modified (diff) |
---|
comment:11 Changed 12 years ago by
Triage Stage: | Accepted → Ready for checkin |
---|
comment:12 Changed 12 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
This was addressed independently in r10727 by just not dropping the admin line from the list. If it turns out people get confused trying to access admin before they've created the views to match what they've put in urls.py then we can add a note about that later.
Duplicate of #6190.