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

generalize link to PyVideo #157

Merged
merged 1 commit into from
Jan 29, 2019
Merged

generalize link to PyVideo #157

merged 1 commit into from
Jan 29, 2019

Conversation

zachvalenta
Copy link
Contributor

Since PyVideo will add new Flask videos all the time, it seems better to direct user to all Flask videos on PyVideo. If we listed all the good Flask videos from PyVideo, there would be too many, but with just a few we risk the videos aging out (the current PyVideo links from Grinberg are nearly 5 years old)

Since PyVideo will add new Flask videos all the time, it seems better to direct user to all Flask videos on PyVideo. If we listed all the good Flask videos from PyVideo, there would be too many, but with just a few we risk the videos aging out (the current PyVideo links from Grinberg are nearly 5 years old)
@zachvalenta
Copy link
Contributor Author

Comments? Issues? Can we get this merged?

@SiqingYu
Copy link

It seems that the three videos listed are beginner-level tutorials.
PyVideo is great. But IMO this list is curated by human and the items are picked intentionally instead of random search results.

@zachvalenta
Copy link
Contributor Author

Gotcha. Should we then update the videos to avoid them aging out?

@SiqingYu
Copy link

It seems that this repo hasn't been maintained since long.
#150

@zachvalenta
Copy link
Contributor Author

Aha, cheers! Didn't notice the larger discussion happening, thanks for the heads up.

@humiaozuzu
Copy link
Owner

Thank you for your help.

@humiaozuzu humiaozuzu merged commit 7583a87 into humiaozuzu:master Jan 29, 2019
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