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(docs): fix player configuration code in drm config tutorial #5359

Merged
merged 1 commit into from
Jun 26, 2023

Conversation

vanyaxk
Copy link
Contributor

@vanyaxk vanyaxk commented Jun 26, 2023

This PR updates the DRM configuration tutorial.

The configuration code example is incorrect, and also it suggests you can change the mapping, which is not true, you are only able to overwrite parts of it.

@google-cla
Copy link

google-cla bot commented Jun 26, 2023

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@avelad avelad added the type: docs Improvements or fixes to documentation label Jun 26, 2023
@avelad avelad added this to the v4.4 milestone Jun 26, 2023
@avelad
Copy link
Member

avelad commented Jun 26, 2023

@vanyaxk Please, sign the CLA! Thanks!

@vanyaxk
Copy link
Contributor Author

vanyaxk commented Jun 26, 2023

done, cheers

@avelad avelad requested review from joeyparrish and theodab June 26, 2023 13:26
@github-actions
Copy link
Contributor

Incremental code coverage: No instrumented code was changed.

@joeyparrish joeyparrish merged commit 5487236 into shaka-project:main Jun 26, 2023
joeyparrish pushed a commit that referenced this pull request Jul 20, 2023
This PR updates the DRM configuration tutorial.

The configuration code example is incorrect, and also it suggests you
can change the mapping, which is not true, you are only able to
overwrite parts of it.

Co-authored-by: Ivan Kohut <[email protected]>
joeyparrish pushed a commit that referenced this pull request Jul 20, 2023
This PR updates the DRM configuration tutorial.

The configuration code example is incorrect, and also it suggests you
can change the mapping, which is not true, you are only able to
overwrite parts of it.

Co-authored-by: Ivan Kohut <[email protected]>
@github-actions github-actions bot added the status: archived Archived and locked; will not be updated label Aug 25, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 25, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status: archived Archived and locked; will not be updated type: docs Improvements or fixes to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants