Opened 16 years ago

Closed 16 years ago

Last modified 13 years ago

#8590 closed (fixed)

Add some information in comment upgrade guide

Reported by: Julien Phalip Owned by: Jacob
Component: Documentation Version: dev
Severity: Keywords:
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

Some users seems to have trouble upgrading to the newcomments contrib app, as in #8571 and http://groups.google.com/group/django-users/browse_thread/thread/eb5db61766a5dc07

Also, as I was going to upgrade to the new system, I jumped straight away to the "Upgrade" page. Then, running the SQL command wouldn't work because I had forgotten to run syncdb to create the new tables. I think this might potentially be a trap for a lot of users.

The attached patch attempts to fix those two issues.

Attachments (2)

8590.comment-upgrade-doc.diff (1.1 KB ) - added by Julien Phalip 16 years ago.
8590.comment-upgrade-doc-2.diff (860 bytes ) - added by Julien Phalip 16 years ago.

Download all attachments as: .zip

Change History (7)

by Julien Phalip, 16 years ago

comment:1 by Malcolm Tredinnick, 16 years ago

Up to Jacob to decide if he wants to include this in general, but you can fix at least part of the patch: we never include information about "if you are running from subversion". The documentation is written for people using a release and that sort of very temporal change will rapidly go out of date. If you're running from subversion you're expected to know how to debug problems of that nature, or at least know enough to check on the mailing list.

by Julien Phalip, 16 years ago

comment:2 by Julien Phalip, 16 years ago

Thanks Malcolm, I've updated the patch following your recommendations and removed the note about subversion.

comment:3 by Jacob, 16 years ago

Owner: changed from nobody to Jacob
Status: newassigned
Triage Stage: UnreviewedReady for checkin

comment:4 by Russell Keith-Magee, 16 years ago

Resolution: fixed
Status: assignedclosed

(In [8652]) Fixed #8590: Added clarification that you need to install the new comments system before trying to port data. Thanks to julien for the report.

comment:5 by Jacob, 13 years ago

milestone: 1.0

Milestone 1.0 deleted

Note: See TracTickets for help on using tickets.
Back to Top