-
Notifications
You must be signed in to change notification settings - Fork 65
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
Conversation
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.
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. |
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.
Tested and passed
- Share option works on Desktop from expanded view
- Leave Call option embedded in the post is now working
- Did a click-through test to ensure all options in the modal and the expanded view are functional on desktop
[MM-62241] Fix screen sharing from Calls popout window desktop#3258 (review)
LGTM!
@streamer45 we could merge this now and open a new issue on the off chance Desktop |
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. |
* Fix screen sharing from popout on new Desktop versions * Fix linting * Fix leaving call from call post in main desktop view
* 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
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