-
Notifications
You must be signed in to change notification settings - Fork 358
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: update internal media core version #3806
Conversation
It would be great to add section with test cases that change works |
This pull request is automatically being deployed by Amplify Hosting (learn more). |
Yeah, I thought the template asked to list out the test cases. Would be good to inform the SDK what we've tested, both manually and with unit tests. |
Please resolve the merge conflicts before merge |
# Conflicts: # packages/@webex/media-helpers/package.json # packages/@webex/plugin-meetings/package.json # packages/calling/package.json # yarn.lock
Co-authored-by: evujici <evujici@cisco.com>
Co-authored-by: evujici <evujici@cisco.com>
COMPLETES SPARK-557155
This pull request addresses
It has been noticed that there was an issue with the
isRequested
flag for transcoded meetings. It was updated ininternal-media-core
, and therefore theinternal-media-core
version needs to be updated here as well to reflect the changes.https://sqbu-github.cisco.com/WebExSquared/webrtc-media-core/pull/264
Change Type
The following scenarios where tested
internal-media-core
for the tweakinternal-media-core
locally with the SDK to ensure it works properlyI certified that
I have read and followed contributing guidelines
I discussed changes with code owners prior to submitting this pull request
I have not skipped any automated checks
All existing and new tests passed
I have updated the documentation accordingly