Opened 10 years ago

Closed 10 years ago

Last modified 7 years ago

#8590 closed (fixed)

Add some information in comment upgrade guide

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


Some users seems to have trouble upgrading to the newcomments contrib app, as in #8571 and

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 10 years ago.
8590.comment-upgrade-doc-2.diff (860 bytes) - added by Julien Phalip 10 years ago.

Download all attachments as: .zip

Change History (7)

Changed 10 years ago by Julien Phalip

comment:1 Changed 10 years ago by Malcolm Tredinnick

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.

Changed 10 years ago by Julien Phalip

comment:2 Changed 10 years ago by Julien Phalip

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

comment:3 Changed 10 years ago by Jacob

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

comment:4 Changed 10 years ago by Russell Keith-Magee

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 Changed 7 years ago by Jacob

milestone: 1.0

Milestone 1.0 deleted

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