calzoneman / sync

Node.JS Server and JavaScript/HTML Client for synchronizing online media
Other
1.47k stars 232 forks source link

Streamable.com support? #585

Closed cobrafast closed 8 years ago

cobrafast commented 8 years ago

Can we get streamable.com support?

Adding the post-URL doesn't work, but adding the video-URL does.

calzoneman commented 8 years ago

I took a look at their API and it seems reasonable, but does not include the duration of the video in the response, which is one of the metadata fields CyTube needs. I've reached out to them via email to ask if this can be added, and if it is then I'll be willing to support streamable.

photonman62 commented 8 years ago

As a workaround, Streamable has direct links to the videos. Right-click the video, click "copy video URL" and you can use that as a synchtu.be link. It doesn't show a title this way, but you can use it for individual videos you want to show on your channel.

calzoneman commented 8 years ago

"Adding the post-URL doesn't work, but adding the video-URL does." -- the person who filed this issue was already aware of this workaround, but it's not ideal for several reasons:

So yes, it is technically possible to use streamable.com videos right now, but if I can hear back from them about including the duration in the JSON API, I'd rather add full support for it.

calzoneman commented 8 years ago

Haven't heard back from them yet, will re-send my email.

calzoneman commented 8 years ago

Email thread quoted below. If I still don't get a response I'll ping the thread again.

Hi,

To give some context, I run a website that allows users to embed videos from various websites and synchronize them so that everyone is watching at the same time. One user requested streamable support via this GitHub Issue: https://github.com/calzoneman/sync/issues/585. In order to set the synchronization timer, as well as display stats such as the total duration of the playlist for a channel, it's necessary for the backend to know how long each video is.

Hopefully this makes the request more clear.

Thanks, Calvin

On 07/13/2016 07:26 PM, Streamable Support wrote:

Hey Calvin,

What specifically do you need the duration for? We don’t show the duration on Streamable.com, and generally try to keep things similarly simple with the API, so I’d need a little more justification before adding this to the to-do list.

Armen

On Jul 13, 2016, at 10:17 PM, Calvin Montgomery wrote:

One last try, from a different address in case my email was getting lost in your spam filter.

Thanks, Calvin

On 07/10/2016 11:00 AM, Calvin Montgomery wrote:

Hello,

Did you get a chance to look at this yet?

Thanks, Calvin

On 06/30/2016 10:33 PM, Calvin Montgomery wrote:

Hello,

I was taking a look at your retrieve API and noticed it does not return the duration of the video. This is useful information, so would it be possible to include in the response?

Thanks, Calvin

lawlmart commented 8 years ago

Duration is now included in the files portion of the API response.

Usual caveats apply - this is not publicly documented and therefore subject to change.

Xaekai commented 8 years ago

That is awesome news.

calzoneman commented 8 years ago

Metadata query implemented: https://github.com/CyTube/mediaquery/pull/5

calzoneman commented 8 years ago

This is now merged, so closing the issue. It will be included the next time cytu.be is deployed.