-
Notifications
You must be signed in to change notification settings - Fork 389
Day 46 Django urls.py import include #10
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
Thanks @ericchou1 is that in the video only? I do see it in the demo code: https://github.com/talkpython/100daysofweb-with-python-course/blob/master/days/045-048-django-intro/demo/mysite/urls.py |
Yes, video only, I went back to the source code to see where include() should have been imported from, then saw it was django.urls. |
Thanks for all the QA Eric. :)
…On Wed, May 29, 2019 at 9:23 AM Eric Chou ***@***.***> wrote:
Yes, video only, I went back to the source code to see where include()
should have been imported from, then saw it was django.urls.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#10?email_source=notifications&email_token=AAPQ62LNFIJRG5U6WNVHA2TPX2UWJA5CNFSM4HQM6WU2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWP4AJQ#issuecomment-497008678>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAPQ62MTPSDG5YTETUFLRRTPX2UWJANCNFSM4HQM6WUQ>
.
|
Of course, thank you @bbelderbos @mikeckennedy and @hobojoe1848 for the great course. Learning a lot! |
@ericchou1 I just checked " URL routing and your first view" and the |
You are right, I must have missed it in the first minute of the video. I will resolve this, thanks for checking! |
Just a note, by default project urls.py did not import include method (at least for me when following the steps) so it needs to be explicitly imported.
Default:
from django.urls import path
Add:
from django.urls import path, include
The text was updated successfully, but these errors were encountered: