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

[stable24] Fix leaving call when switching to another conversation #8530

Merged
merged 2 commits into from Jan 5, 2023

Conversation

backportbot-nextcloud[bot]
Copy link

backport of #8519

When a conversation was left in the Files app the store was cleared to
ensure that it did not contain old data if the conversation was joined
again later (as the conversation data will not be updated while not in
the conversation). However, as this is only needed in case the
conversation is joined again this can be also done before actually
joining the conversation.

This change will be used to automatically leave the call when leaving
the conversation; leaving the call needs to get some values from the
store that were cleared, so this change avoids having to explicitly wait
until the conversation was left before clearing the store.

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
When a conversation is joined while being in another conversation the
previous conversation is left. Leaving a conversation automatically
leaves the call in the server, but in the WebUI it had no effect on the
call. Therefore, from the point of view of the WebUI the user was still
in the call in the previous conversation, so switching back to it caused
the call view to be shown again (although the call was not actually
joined again, only the call view was shown).

To solve that now an ongoing call is automatically left when its
conversation is left.

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
@nickvergessen nickvergessen merged commit 7ed2712 into stable24 Jan 5, 2023
@nickvergessen nickvergessen deleted the backport/8519/stable24 branch January 5, 2023 13:50
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.

None yet

2 participants