Code

Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#11959 closed (fixed)

In tutorial, admin is broken throughout most of part 3

Reported by: carljm Owned by: nobody
Component: Documentation Version: 1.1
Severity: Keywords:
Cc: Triage Stage: Unreviewed
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:

Description

Now that the admin uses url-reversing, it can't handle a urlconf that refers to nonexistent view functions. Part 3 of the tutorial instructs users to write out four URL patterns for nonexistent views, and then slowly fills in the associated views. The last of the four is not created until midway through part 4. This means that new users who have just completed part two of the tutorial and are all excited about their new admin interface now find it mysteriously broken with ViewDoesNotExist errors throughout almost all of part 3 and the beginning of part 4, with no explanation or reference to this in the tutorial text.

The simplest solution is to write placeholder views right away for all four URL patterns. The attached patch does this (along with a couple other very minor tweaks in tutorial parts 2 and 3). The other alternative would be to instruct the user to comment out several of the patterns until their associated views are written, but this seemed to require more changes and more convoluted language.

Attachments (1)

11959_r11594.diff (2.6 KB) - added by carljm 5 years ago.
patch to fix admin in tutorial part 3, plus two minor tweaks

Download all attachments as: .zip

Change History (3)

Changed 5 years ago by carljm

patch to fix admin in tutorial part 3, plus two minor tweaks

comment:1 Changed 5 years ago by russellm

  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset
  • Resolution set to fixed
  • Status changed from new to closed

(In [11621]) Fixed #11959 -- Updated the tutorial to ensure that the admin site continues to work after URLpatterns are introduced. Thanks to carljm for the report and draft patch.

comment:2 Changed 5 years ago by russellm

(In [11622]) [1.1.X] Fixed #11959 -- Updated the tutorial to ensure that the admin site continues to work after URLpatterns are introduced. Thanks to carljm for the report and draft patch.
Backport of r11621 from trunk.

Add Comment

Modify Ticket

Change Properties
<Author field>
Action
as closed
as The resolution will be set. Next status will be 'closed'
The resolution will be deleted. Next status will be 'new'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.