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

Support for call upgrades #1827

Merged
merged 9 commits into from
Sep 21, 2021

Conversation

SimonBrandner
Copy link
Contributor

@SimonBrandner SimonBrandner commented Aug 4, 2021

Type: enhancement
Requires matrix-org/matrix-react-sdk#6547


Notes: Support for call upgrades. setLocalVideoMuted() and setMicrophoneMuted() are now async and return the new mute state


Here's what your changelog entry will look like:

🚨 BREAKING CHANGES

  • Support for call upgrades. setLocalVideoMuted() and setMicrophoneMuted() are now async and return the new mute state (#1827). Contributed by SimonBrandner.

@SimonBrandner SimonBrandner changed the title Support upgrading voice calls to video calls Support for call upgrades Sep 18, 2021
@SimonBrandner SimonBrandner marked this pull request as ready for review September 18, 2021 10:46
@SimonBrandner SimonBrandner requested a review from a team as a code owner September 18, 2021 10:46
Copy link
Member

@turt2live turt2live left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

otherwise looks good! Not sure if we want to squeeze this into the RC or if we should delay it. Very keen not to stick it into the release just because we have another breaking change: version numbers are relatively free, and we can afford back-to-back breaking releases.

src/webrtc/call.ts Outdated Show resolved Hide resolved
src/webrtc/call.ts Outdated Show resolved Hide resolved
@SimonBrandner
Copy link
Contributor Author

SimonBrandner commented Sep 19, 2021

otherwise looks good! Not sure if we want to squeeze this into the RC or if we should delay it. Very keen not to stick it into the release just because we have another breaking change: version numbers are relatively free, and we can afford back-to-back breaking releases.

Yeah, makes sense, let's wait a bit 👍

Copy link
Member

@turt2live turt2live left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

otherwise looks good.

and yes, we'll be waiting until after the RC on this.

src/webrtc/call.ts Show resolved Hide resolved
Signed-off-by: Šimon Brandner <[email protected]>
Copy link
Member

@turt2live turt2live left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks :)

@turt2live turt2live merged commit b53566e into matrix-org:develop Sep 21, 2021
@SimonBrandner SimonBrandner deleted the feature/call-upgrades branch September 21, 2021 15:10
su-ex added a commit to SchildiChat/matrix-js-sdk that referenced this pull request Oct 16, 2021
* Support for call upgrades. `setLocalVideoMuted()` and `setMicrophoneMuted()` are now `async` and return the new mute state ([\matrix-org#1827](matrix-org#1827)). Contributed by [SimonBrandner](https://github.com/SimonBrandner).
* Implement file versioning for tree spaces ([\matrix-org#1952](matrix-org#1952)).
* Allow answering calls without audio/video ([\matrix-org#1950](matrix-org#1950)). Contributed by [SimonBrandner](https://github.com/SimonBrandner).
* Add `bound` to `IThreepid` ([\matrix-org#1941](matrix-org#1941)). Contributed by [SimonBrandner](https://github.com/SimonBrandner).
* Add `trusted_locally` to `TrustInfo` ([\matrix-org#1942](matrix-org#1942)). Contributed by [SimonBrandner](https://github.com/SimonBrandner).
* Fix incorrect return value type in getJoinedRooms() ([\matrix-org#1959](matrix-org#1959)). Contributed by [psrpinto](https://github.com/psrpinto).
* Make sure to set `callLengthInterval` only once ([\matrix-org#1958](matrix-org#1958)). Fixes element-hq/element-web#19221 and element-hq/element-web#19221. Contributed by [SimonBrandner](https://github.com/SimonBrandner).
* Fix event partitioning from non threading ready clients ([\matrix-org#1948](matrix-org#1948)).
* Ensure unencrypted fields get exposed by getEffectiveEvent() ([\matrix-org#1938](matrix-org#1938)). Fixes element-hq/element-web#19062 and element-hq/element-web#19062.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants