Skip to content

Conversation

@FoliniC
Copy link

@FoliniC FoliniC commented Mar 22, 2020

I'm quite new to django... so it takes some googling to know that I should add the import to use url()

I'm quite new to django... so it takes some googling to know that I should add the import to use url()
@coveralls
Copy link

Coverage Status

Coverage remained the same at 88.389% when pulling 45d1601 on FoliniC:patch-1 into 2147d3d on dokterbob:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 88.389% when pulling 45d1601 on FoliniC:patch-1 into 2147d3d on dokterbob:master.

#) Import subscription, unsubscription and archive URL's somewhere in your
`urls.py`::
from django.conf.urls import url

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

... to indicate that there is likely stuff in between

@dokterbob dokterbob closed this Oct 24, 2020
@dokterbob
Copy link
Collaborator

@FoliniC Thanks for the PR. But the proposed code is already in Django's autogenerated urls.py so it's really hard to miss and it will create mental noise for anyone but very novice users.

Hope you're having a good time learning!

Thanks for understanding!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants