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

fix(FEC-11090): PLAY/PAUSE button got stuck in play IMA + DAI while IMA DAI ad playing. #64

Merged
merged 2 commits into from
Mar 16, 2021

Conversation

Yuvalke
Copy link
Contributor

@Yuvalke Yuvalke commented Mar 16, 2021

Description of the Changes

Issue: plugin ask the engine too early and gets paused response for the video tag while IMA playing and the video tag is actually paused.
Solution: delay the check until ad event actually needs to dispatch - IMA DAI playing

CheckLists

  • changes have been done against master branch, and PR does not conflict
  • new unit / functional tests have been added (whenever applicable)
  • test are passing in local environment
  • Travis tests are passing (or test results are not worse than on master branch :))
  • Docs have been updated

…MA DAI ad playing.

Issue: plugin ask the engine too early and get paused response for video tag while IMA playing and the video tag is actually paused.
Solution: delay the check until ad event actually needs to dispatch - IMA DAI playing
@Yuvalke Yuvalke requested a review from a team March 16, 2021 12:54
@Yuvalke Yuvalke self-assigned this Mar 16, 2021
yairans
yairans previously approved these changes Mar 16, 2021
@Yuvalke Yuvalke merged commit df09a8c into master Mar 16, 2021
@Yuvalke Yuvalke deleted the FEC-11090 branch March 16, 2021 13:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants