-
Notifications
You must be signed in to change notification settings - Fork 174
VPAID integration #50
base: master
Are you sure you want to change the base?
Conversation
Also if anyone has examples of non-linear javascript VPAID ad tags I would be happy to check and try to implement in my spare time |
dailymotion/vast-client-js#52 was merged so I've updated code to use latest |
Thanks for the PR! Let me take a look at this, and I'll try to get it merged in the next couple of days. |
vpaid tests
…blem with videojs-youtube in FF) check correct tag even on ios and ipad
Any luck with checking? |
Hey, @mente, unfortunately, we just moved some tests and implementation code around, which kinda complicates the merge here. I'm gonna try to merge this one on the command line and take a look. I should know more later this afternoon, thanks for your patience! |
Any help needed? |
I think I'm OK for right now—it's just that there's a whole lot of code here. This PR almost doubles the size of the videojs.vast.js file, so I want to make sure that I'm fully understanding the changes in here. |
I was thinking of making it as another plugin but decided to make it in 2014-11-03 18:49 GMT+02:00 Chris Sinchok [email protected]:
|
I think that fine, but I'm leaning towards doing some refactoring, and separating the VPAID logic into a separate file in this repo. Right now there's a just a whole lot going on in this one file. |
Maybe I should open another PR against current master? It can speed up process of merging |
I'm also waiting for that merge to happen : ) I would love a VPAID support! |
Hey guys, I'm using this plugin in my VAST-only custom player and I'd love to see support for this cool plugin too! |
any updates on this pull request? Anything the open-source community can do to help expedite this? |
Waiting for this one, too. Any ETA? |
So... Has VAST 2.0 and VPAID been integrated into this plugin yet? I see there is a merge conflict so it must not be in there yet... Really enjoy VideoJS but with the new ads my company is trying to roll out with YUME I'm waiting around to integrate it. |
@Manbearpixel as far as I know YUME is not compatible with javascript SDKs by default. When I was integrating it in our project had to ask them to leave VAST only. Reason is they have only Flash SDK. While Liverail has both. |
@mente But there are plugins out there such as OVA which fully integrate VPAID support. I was originally going to go that route but the plugin provider seems to now be owned by JWPlayer, which closed their developer portal so you can't easily download their source files. Plus on top of that, I believe you have to pay them to have ads on any player that will use that plugin. I had switched to VideoJS for this new site and it's been fine until our need to support YUME flash ads which use VPAID. |
I receive several questions like "Why this ad tag doesn't work?" 100% of them are based on Flash VPAID SDK. I contact Actionscript developer whether it's possible to painlessly bridge javascript <-> swf SDKs. If yes - will update my PR (or open a new one). |
I did a basic bridge between SWF and Javascript (using external interface) if you are interested I can share the code. |
@leomicheloni would be great! |
I would also be very interested in seeing that code! We are working on |
No problem, i'm uploading it to github in a few hours |
I couldn't find the code. But public class FlashAdapter extends Sprite{
pubic FlashAdapter(){
};
public LocalMethod(params: Object): void{
//this method is called from javascript
}
} You can call javascript function from Action Script using ExternalInterface.call("window.myGlobalFunction", jsonDataToSendToJavascript); And you can listen to calls from javascript ExternalInterface.addCallback( "flashMethid", this,LocalMethod ); So, you can simple load the swf object using swfobject lib Then call the method from javascript var swf = swfobject.flashMethod(params); That's all. If you are interested we can build an adapter from scratch. |
Is this ready to be merged for the VPAID js part of the compatibility? |
@Kalail I too am still waiting for any more feedback about VPAID integration into this plugin. VideoJS is a great player but I am now looking for alternative players that, more than likely, will cost a licensing fee to use. I tried my luck with the VideoJS Google IMA plugin which has VPAID support, unfortunately it does not run the VPAID via Flash and failed. |
Sorry for being long on it, too much events happening in my life. Right now I don't free time for updating it and adding SWF support. If you really need it - contact me in private |
@mente Is it money issue ? if yes send my PM, I have website and VPAID support is critical for my site. |
@Manbearpixel: There has been work done by the guardian around flash vpaid that i've tested and got working: https://github.com/guardian/video-js-vpaid I'm also in the middle of refactoring code around javascript vpaid vast support that I want to contribute back to the community at some point once it's stable. @mente: i'm refactoring your pull request and refactoring it in such a way that javascript vpaid is just another videojs tech. hopefully it works out reasonably well. |
@pcting Oh, thanks! I had searched all over I thought for any mention of VideoJS + VPAID and did not find that fork... This is the exact thing I was working on, I'll head over and check out the project. Cheers - mbp |
@pcting I was looking into the video-js-vpaid repo, went through the steps to get the sandbox working and it's not running. Would you be able to provide assistance? My email is: [email protected] Thanks in advance. |
@pcting hi, I also tried the video-js-vpaid repo and I get an error about "Vpaid" tech being undefined in video.dev.js when I was testing the /sandbox/videojs.html. Did you write you own plugin for the vpaid tech or did it work with the current video.dev.js? Thanks! |
@eccheung4 I just put together a forked repo of the guardians' VPAID attempt. My repo is a compilation of their's and some content @pcting threw my way. At this moment I do have some sort of working demonstration of Flash VPAID support. A LiveRail VPAID ad test is working, but a YuMe one is not for some reason (could be on their end). Anyways, check it out and run the installation on my repo and it should work fine for you. I haven't touched much since I worked on it a day or so ago since I'm waiting for YuMe to reply back with an issue. |
Cool, thanks @Manbearpixel ! I will check it out |
This is interesting progress. I have a need for Flash VPAID support for Video.js in a timely manner and am willing to offer a monetary reward, pending a requirements review. |
@eccheung4 I'm in the process of cleaning up the code to put out a proper release with an actual working example. I am hopeful I can have the code polished and ready within the next two weeks. I unfortunately have rewritten much of the videojs-vast-plugin where I can't see it being merged back upstream. |
<script src="lib/videojs-contrib-ads/videojs.ads.js"></script> | ||
<script src="bower_components/videojs/dist/video-js/video.js"></script> | ||
<script src="bower_components/videojs-contrib-ads/src/videojs.ads.js"></script> | ||
<script src="bower_components/vast-client-js/vast-client.js"></script> | ||
|
||
<script src="lib/vast-client.js"></script> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this still necessary with line 19 using vast-client from bower components?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nope, just missed it
@pcting very cool! Looking forward to it. I forked @Manbearpixel and was able to get LiveRail Flash, Spotx Flash, and Adaptv flash, but I couldn't get YUME working. |
@eccheung4 Good stuff man, I'll check it out. Will be working with YuMe to determine why their integration isn't working as easily as others. |
@Manbearpixel I managed to run yume vpaid with minor changes to AdContainer.as |
awesome @rhekeen ! Unfortunately I tried out your repo and I get a vpaid creative ad error for your yume vpaid test. Is all you did was cast the loaded swf object as a Display Object? Is there anything else that maybe you forgot to commit thats needed? thanks! |
Is this PR getting close to being merged? I'm extremely interested in using this feature.... |
any progress made in May? thx |
@Danovadia @joshmadewell - I'm working on a VPAID plugin for VideoJS currently. I do not have a repo on it yet since I'm still finishing it up but will source it when available. I had a working VPAID integration in the VideoJS Flash SWF but am having issues setting the source back once the ad is over, so instead I'm turning it into a plugin that will overlay the video player. |
@Danovadia @joshmadewell - Finally wrapped up my code into a repo, feel free to check it out. Hope this helps! https://github.com/Manbearpixel/videojs-vpaid |
This is awesome man! Are you using this because you only use VPAID ads or do you have some kind of implementation on your end where you switch plugins based on the tag? |
@joshmadewell |
@Manbearpixel revisiting this PR, does your videojs-vpaid plugin work alongside this one or meant to replace? Curious if you're using both plugins to support both VAST and VPAID ads for your site |
I asked a similar question if you look at the comment just above MBP's last one. He said he implemented a way to know whether or not he needed the VAST or VPAID plugin before choosing which ad tech to use. I needed to be able to know if the ads failed at run-time so i made some modifications to the VPAID code that MBP wrote to handle this which was very simple to do. |
@spra85 |
TL;DR - only Javascript VPAID SDKs are supported as for now. Known VPAID SDKs in Javascript - Liverail (via parameter
LR_FORMAT
, check docsHello,
This PR adds basic support for linear VPAID ads. PR is based on squashed branch. I've kept original branch with all commits in if you like.
Generally what has been done:
Tested on Chrome stable latest, IE9, Firefox stable latest, iOS7.1, iOS8.0, Android 4.3 (Chrome).
PR is pretty big, looking forward to comments and ideas for improvement.
Regards,
Alex