-
Notifications
You must be signed in to change notification settings - Fork 355
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(plugin-meetings): clientSignallingProtocol stat #3803
fix(plugin-meetings): clientSignallingProtocol stat #3803
Conversation
This pull request is automatically being deployed by Amplify Hosting (learn more). |
171ef4e
to
3190a20
Compare
packages/@webex/plugin-meetings/test/unit/spec/meeting/index.js
Outdated
Show resolved
Hide resolved
90d9826
to
fb18e56
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thank you!
fb18e56
to
299fe44
Compare
Co-authored-by: Filip Nowakowski <fnowakow@cisco.com>
Co-authored-by: Filip Nowakowski <fnowakow@cisco.com>
Co-authored-by: Filip Nowakowski <fnowakow@cisco.com>
COMPLETES WEBEX-396966
This pull request addresses
Missing
clientSignallingProtocol
stat in media quality eventby making the following changes
Fill
clientSignallingProtocol
atmeetingJoinInfo.clientSignallingProtocol
with valueWebRTC
Change Type
The following scenarios where tested
I 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
Make sure to have followed the contributing guidelines before submitting.