-
Notifications
You must be signed in to change notification settings - Fork 217
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
SetProgress is avoiding quartileEvents to be fired several times #460
Comments
Hello ! Thanks for raising this issue :) This is an interesting feature to add since it does not modify the initial behavior of the I will look further into your pull request. :) |
Seen the pull request be merged. |
It will be available in the coming weeks :) |
Hello dailymotion team. I came here to get updates. Is there a planned 5.1 release that includes the fix I submitted? |
Hello! A major update will be release in february. You can see the changes on this pull request . The change you submitted has been merged to master and will be available in the next release of the vast-client. :) |
Hello, closing this issue as it has been fixed in 6.0.0 version |
vast-client-js 5.0.0
Using the VPAIDTracker to call event urls.
We have the setProgress call on video progress, this fires quartile automatically wich is convenient.
issue is :
Some of or publishers are using a replay button so the ad can be rewound. setProgress is calling the "rewind" event as expected but then the quatile events are not fired anymore.
This is because the setProgress event is setting "once: true" on track() call which removes the tracking urls from the url array.
To avoid this we trigger quartile urls directly with track() and avoid using setProgress.
can we add a parameter on setProgress ( 'trackOnce' = true ) so anyone can set it to false if needed please ?
made the pull request here : #461
The text was updated successfully, but these errors were encountered: