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

update video_info version to 2.5 which supports Youtube API v3 #76

Closed
wants to merge 1 commit into from

Conversation

0mega
Copy link

@0mega 0mega commented Sep 3, 2015

Bump video_info to 2.6 to solve the issue #69.
It also introduces new functionalities described here

@gottfrois
Copy link
Owner

Don't we need to setup youtube api keys?

@0mega
Copy link
Author

0mega commented Sep 3, 2015

We probably will, but first there are some other issues. It's not working quite well. I think there is still some work to be on video_info. I'll keep you updated.

@0mega
Copy link
Author

0mega commented Sep 4, 2015

Actually it seems that we don't need the API keys, because video_info offers scrapers so it's ok to use it as is.

But this PR won't solve anything, we will have to wait for more 4 issues of video_thumbnailer to be implemented and then update the version to 2.6. The fix for the #69 is already implemented in video_info master.

Cheers

List of 2.6 features can be found here thibaudgg/video_info#103
@0mega 0mega force-pushed the update_video_info_gem branch from ce9bf46 to c529a2b Compare March 5, 2016 10:42
@gottfrois
Copy link
Owner

Issue fixed in v3.1.1

@gottfrois gottfrois closed this Mar 5, 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.

2 participants