Include namespace with the urlpatterns

[dnaranjo89]

Otherwise update the doc to include the namespace (#103):

import notifications

urlpatterns = patterns('',
    ...
    url('^inbox/notifications/', include(notifications.urls, namespace='notifications')),
    ...
)

[yangyubo]

@zhang-z Could you please review this PR, I haven't touch Django for a very long time…

[zhang-z]

Hi @dnaranjo89 , thanks for the update.

The 3-tuple is deprecated since Django 1.9:

Support for the app_name argument is deprecated and will be removed in Django 2.0. Specify the app_name as explained in URL namespaces and included URLconfs instead.

Support for passing a 3-tuple is also deprecated and will be removed in Django 2.0. Pass a 2-tuple containing the pattern list and application namespace, and use the namespace argument instead.

For instance namespace, the more recommended way would be to specify it in include(notifications.urls, namespace='notifications'). For application namespace, in pull request #104, I have added 'app_name' to notifications/urls.py, which will apply to version 1.9 and above. For version 1.8 and below, we can still specify it in include().

[dnaranjo89]

Perfect, thanks for the fast update!