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

Video metadata #6

Open
harvzor opened this issue Sep 13, 2017 · 2 comments
Open

Video metadata #6

harvzor opened this issue Sep 13, 2017 · 2 comments
Milestone

Comments

@harvzor
Copy link

harvzor commented Sep 13, 2017

It might be nice to include methods on the video which allow us to get video metadata such as video thumbnails.

Here is a fairly crude method which I'm using on my project:

public static string GetVideoThumbnail(this Videolizer.VideolizerVideo video)
{
    if (video.type == Videolizer.VideolizerVideo.VideoTypes.Vimeo)
    {
        return "https://i.vimeocdn.com/video/" + video.id + "_640.jpg";
    }
    else if (video.type == Videolizer.VideolizerVideo.VideoTypes.YouTube)
    {
        return "http://img.youtube.com/vi/" + video.id + "/0.jpg";
    }

    return "";
}

It may be better if the APIs of the respective videos are used to ensure that the thumbnails returned are always correct, but I worry this might increase page load time if these methods are relied on synchronously.

Perhaps an even better idea would be to use the APIs when the video is added in Umbraco so all of the extra metadata can be retrieved and saved to the node.

@dinc5150
Copy link
Collaborator

Once I get the search feature tested and released, I'm planning on grabbing a lot more meta info at the time that its added which I will store in the property value and create methods to pull them out.

@dinc5150 dinc5150 added this to the V2.1 milestone Sep 14, 2017
@dinc5150
Copy link
Collaborator

Note to self: I've also had a request to store the Timecode from the query string, which I think is a good idea.

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

No branches or pull requests

2 participants