Skip to content
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

Development2 #13

Merged
merged 27 commits into from Aug 10, 2016
Merged

Development2 #13

merged 27 commits into from Aug 10, 2016

Conversation

ghoshbishakh
Copy link
Collaborator

No description provided.

@jchoude
Copy link
Collaborator

jchoude commented Jul 22, 2016

Hi @ghoshbishakh do you plan on adding anything, or I can go ahead and check it?

@@ -20,3 +20,6 @@
url(r'^admin/', admin.site.urls),
url(r'^', include('website.urls')),
]

handler404 = 'website.views.custom404'
Copy link
Collaborator

Choose a reason for hiding this comment

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

For my info, those do not seem to be added to url_patterns. Are those names (handler404 and 500) reserved words?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes handler404 and handler500 are reserved words. Django performs some kind of magic with them and renders the views assigned to them in case of 404 or 500.

@jchoude
Copy link
Collaborator

jchoude commented Jul 22, 2016

Overall, seems good.

@ghoshbishakh
Copy link
Collaborator Author

@jchoude So should I push future changes in a different PR?

@ghoshbishakh
Copy link
Collaborator Author

@Garyfallidis @jchoude I am not pushing any more commits in this PR. So this may be reviewed now.

Channel ID of the youtube channel for which the videos will
be retrieved.

count: int
Copy link
Contributor

Choose a reason for hiding this comment

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

count : int

@Garyfallidis Garyfallidis merged commit 60778b0 into dipy:master Aug 10, 2016
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.

None yet

3 participants