Opened 16 years ago

Closed 15 years ago

Last modified 13 years ago

#9675 closed (fixed)

Docs about migrating from the old comment system: add note about urls.py

Reported by: Maciej Bliziński Owned by: Jacob
Component: Documentation Version: 1.0
Severity: Keywords:
Cc: Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

It's worth mentioning that the entry from urls.py:

(r'^comments/', include('django.contrib.comments.urls.comments')),

Needs to be changed to:

(r'^comments/', include('django.contrib.comments.urls')),

Looking at #8571 shows that many poeple have had this problem.

Change History (10)

in reply to:  description comment:1 by Maciej Bliziński, 16 years ago

comment:2 by Jacob, 15 years ago

milestone: 1.1
Triage Stage: UnreviewedAccepted

comment:3 by Jacob, 15 years ago

Owner: changed from nobody to Jacob
Status: newassigned

comment:4 by Idan Gazit, 15 years ago

Resolution: fixed
Status: assignedclosed

Looks like this was solved in r8557. Cannot find any references to (r'^comments/', include('django.contrib.comments.urls.comments')) anywhere in the docs; closing.

comment:5 by Maciej Bliziński, 15 years ago

Resolution: fixed
Status: closedreopened

idangazit, the lack of mentions of "(r'comments/', ..." in the docs is the original problem reported here.

comment:6 by Idan Gazit, 15 years ago

That seems to be doubly incorrect, unless I'm missing something large here.

The issue reported above is that the module path for including the comments urlconf was incorrect. I've grepped through the docs and found that problem fixed in r8557.

The new issue that you bring up ("lack of mentions of the correct way to include urlconf") doesn't seem to be true, the correct urlconf gets prominent placement at the top of the comments docs in the Quick Start Guide, and again in the documentation on Including Other URLconfs.

I don't see a natural place in any of the other comments docs for a re-mentioning of the URLconf. I think that the comments docs could use some expansion but barring that, don't know what to do with this ticket.

comment:7 by Maciej Bliziński, 15 years ago

If I understand correctly, the issue reported is:

"There is a bit of information that is not there in the docs, and it should be there."

In other words, there is something to add to the migration documentation. The bit to be added is (in quotes):

"""The entry from urls.py:

(r'^comments/', include('django.contrib.comments.urls.comments')),

Needs to be changed to:

(r'^comments/', include('django.contrib.comments.urls')),

"""

comment:8 by Jacob, 15 years ago

Status: reopenedassigned

comment:9 by Jacob, 15 years ago

Resolution: fixed
Status: assignedclosed

(In [10746]) Fixed #9675: added note about upgrading the URLconf to the comment upgrade guide.

comment:10 by Jacob, 13 years ago

milestone: 1.1

Milestone 1.1 deleted

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