-
Notifications
You must be signed in to change notification settings - Fork 6.7k
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(config): Remove the deprecated configs. #13500
Conversation
Add the new codec selection settings for desktop and mobile endpoints.
5250288
to
042936a
Compare
@@ -47,13 +47,11 @@ const INITIAL_RN_STATE: IConfig = { | |||
// FIXME: than requiring this override here... | |||
|
|||
p2p: { | |||
disabledCodec: 'vp9', | |||
preferredCodec: 'h264' | |||
mobileCodecPreferenceOrder: [ 'H264', 'VP8' ] |
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.
What the value if we don't define this? We could make the code behave like so, and remove this override entirely, and people can configure whatever they want in config.js, no?
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.
https://github.com/jitsi/lib-jitsi-meet/pull/2296/files#diff-dcb81f0be0b559eee3d7693e162f67f90eee41d6fa0a8cbc91c6460ec02f5c36R38 will be the new default order which is applicable to desktop clients. We could define a different set for mobile endpoints by default.
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.
Let's do that please!
Other PRs seem to pass. Are we sure the failures are not related? |
MuteTest on Firefox has been failing on alpha for few weeks now so that one is unrelated for sure. |
741fa17
to
9fc097b
Compare
* fix(config): Remove the deprecated configs. Add the new codec selection settings for desktop and mobile endpoints. * fix(config): Use the new codec selection settings for RN. * Address review comments. * chore(deps) Update lib-jitsi-meet@latest. jitsi/lib-jitsi-meet@v1649.0.0+17ade96a...v1652.0.0+90da4884.
Add the new codec selection settings for desktop and mobile endpoints.