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

[stable19] Ignore "forceMute" message if already muted #4164

Merged
merged 3 commits into from Sep 16, 2020

Conversation

backportbot-nextcloud[bot]
Copy link

@backportbot-nextcloud backportbot-nextcloud bot commented Sep 16, 2020

⚠️ This backport had conflicts and is incomplete ⚠️ Conflicts were due to changes in code not found in stable19, so the backport is fine.

backport of #4076

Before the UI only allowed to send the "forceMute" message to
participants that were not muted. Now that moderators can mute all the
other participants at once the message may be received by participants
that are already muted, so it needs to be ignored in that case.
Otherwise the user would see a "You have been muted by a moderator"
notification even if already muted.

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
Copy link
Member

@danxuliu danxuliu 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 works 👍

@danxuliu danxuliu merged commit c91791a into stable19 Sep 16, 2020
@danxuliu danxuliu deleted the backport/4076/stable19 branch September 16, 2020 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug feature: call 📹 Voice and video calls feature: frontend 🖌️ "Web UI" client
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant