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

[MM-62241] Fix screen sharing from popout on new Desktop versions #948

Merged
merged 6 commits into from
Jan 20, 2025

Conversation

streamer45
Copy link
Collaborator

Summary

We need some plugin changes to make mattermost/desktop#3258 work. We'll need to backport this fix in earlier plugin versions to prevent the bug when users upgrade desktop version.

Ticket Link

https://mattermost.atlassian.net/browse/MM-62241

@streamer45 streamer45 added 2: Dev Review Requires review by a core committer 2: QA Review Requires review by a QA tester labels Jan 20, 2025
@streamer45 streamer45 added this to the v1.5.1 milestone Jan 20, 2025
@streamer45 streamer45 self-assigned this Jan 20, 2025
Copy link
Member

@cpoile cpoile left a comment

Choose a reason for hiding this comment

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

Does this mean we're removing the backwards compatibility?

@streamer45
Copy link
Collaborator Author

Does this mean we're removing the backwards compatibility?

@cpoile It's getting removed from Desktop since we had a couple of releases already with the new API, which is what caused the issue to surface. But this is a breaking change because we failed to update the plugin side in time. In popout we were still relying on the legacy (deprecated) API.

Copy link

@DHaussermann DHaussermann left a comment

Choose a reason for hiding this comment

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

Tested and passed

LGTM!

@streamer45 streamer45 added 3: Reviews Complete All reviewers have approved the pull request and removed 2: Dev Review Requires review by a core committer 2: QA Review Requires review by a QA tester labels Jan 20, 2025
@DHaussermann
Copy link

@streamer45 we could merge this now and open a new issue on the off chance Desktop v5.11.0-rc1 presents any new problems.

@streamer45
Copy link
Collaborator Author

@streamer45 we could merge this now and open a new issue on the off chance Desktop v5.11.0-rc1 presents any new problems.

I am waiting to fix e2e tests (#950). I am also planning a couple of Calls dot releases, namely v1.5.1 and v0.29.7 to include this fix since these functionalities will break when using the new desktop version against older plugins.

@streamer45 streamer45 merged commit e0941cc into main Jan 20, 2025
21 checks passed
@streamer45 streamer45 deleted the MM-62241 branch January 20, 2025 22:04
streamer45 added a commit that referenced this pull request Jan 21, 2025
* Fix screen sharing from popout on new Desktop versions

* Fix linting

* Fix leaving call from call post in main desktop view
@streamer45 streamer45 mentioned this pull request Jan 21, 2025
streamer45 added a commit that referenced this pull request Jan 21, 2025
* Skip global widget window from available screen sharing sources (#930)

* Fix panic in handleBotGetProfileForSession (#942)

* [MM-62241] Fix screen sharing from popout on new Desktop versions (#948)

* Fix screen sharing from popout on new Desktop versions

* Fix linting

* Fix leaving call from call post in main desktop view

* Bump transcriber and recorder images
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3: Reviews Complete All reviewers have approved the pull request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants