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

[Maciej Bliziński]

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.

[Maciej Bliziński]

The mentioned documentation page:
​http://docs.djangoproject.com/en/dev/ref/contrib/comments/upgrade/

[Idan Gazit]

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.

[Maciej Bliziński]

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

[Idan Gazit]

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.

[Maciej Bliziński]

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')),

"""

[Jacob]

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

[Jacob]

Milestone 1.1 deleted