Navigation Menu

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

[JW8-12180] Updating captions logic to only use native captions while casting #3978

Merged
merged 2 commits into from Feb 10, 2022

Conversation

Dawolee
Copy link
Contributor

@Dawolee Dawolee commented Feb 8, 2022

This PR will...

Update the captions logic to use native captions when casting.

Why is this Pull Request needed?

Duplicate caption tracks were being added when casting.

Are there any points in the code the reviewer needs to double check?

Are there any Pull Requests open in other repos which need to be merged with this?

https://github.com/jwplayer/jwplayer-commercial/pull/8316

Addresses Issue(s):

JW8-12180

Checklist

  • Jenkins builds and unit tests are passing
  • I have reviewed the automated results

@jwplayer-robot
Copy link

MULTI Build for commit 0c259a4 did not complete (FAILURE).
🏗️▪️ jwplayer build SUCCESS
📋▪️ jwplayer unit tests FAILURE
🏗️▪️ jwplayer-commercial build NOT_STARTED
📋▪️ jwplayer-commercial unit tests NOT_STARTED
🥒 Cucumber Integration Tests NOT_STARTED
🎃 Manual Tests
📺 Views

@Dawolee
Copy link
Contributor Author

Dawolee commented Feb 10, 2022

closing and re-opening this on new mono-repo

@Dawolee Dawolee closed this Feb 10, 2022
@Dawolee Dawolee reopened this Feb 10, 2022
@Dawolee Dawolee merged commit c280de1 into master Feb 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants