Opened 14 years ago
Closed 14 years ago
#17490 closed Cleanup/optimization (fixed)
A few minor typos
| Reported by: | Owned by: | nobody | |
|---|---|---|---|
| Component: | Documentation | Version: | 1.3 |
| Severity: | Normal | Keywords: | tutorial |
| Cc: | Triage Stage: | Ready for checkin | |
| Has patch: | no | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | yes | UI/UX: | no |
Description
I noticed a couple minor typos while going through the tutorial. They probably won't trip up a seasoned programmer, but likely to be a problem for a complete newbie.
- When adding polls to INSTALLED_APPS, the comma is missing after it - https://docs.djangoproject.com/en/1.3/intro/tutorial01/
- Suggestion to add a prompt to run syncdb again after "Make poll app modifiable" section. https://docs.djangoproject.com/en/1.3/intro/tutorial02/
I think the tutorial is great! I am still pretty new to programming altogether and found the format, explanations, hyperlinks (to read more about a particular topic), etc to be very, very, very helpful. Looking forward to diving into the rest.
Change History (2)
comment:1 by , 14 years ago
| Component: | Uncategorized → Documentation |
|---|---|
| Triage Stage: | Unreviewed → Ready for checkin |
| Type: | Uncategorized → Cleanup/optimization |
Note:
See TracTickets
for help on using tickets.
Replying to malcolm.newsome@…:
Yes, while the current code is correct, it's still a good idea to include a trailing comma, to avoid being tripped up by Python's automatic string concatenation.
This instruction is given in the "Activate the admin site", just after adding
"django.contrib.admin"toINSTALLED_APPS. There's no reason to mention it again, is there?Thanks for your feedback!