Skip to content

Leave call and room when logging out - #1929

Merged
nickvergessen merged 1 commit into
masterfrom
bugfix/1872/leave-call-and-room-when-logging-out
Jul 3, 2019
Merged

Leave call and room when logging out#1929
nickvergessen merged 1 commit into
masterfrom
bugfix/1872/leave-call-and-room-when-logging-out

Conversation

@nickvergessen

Copy link
Copy Markdown
Member

Fix #1872

Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen

Copy link
Copy Markdown
Member Author

/backport to stable16

@danxuliu danxuliu left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Tested (with and without MCU) and works 👍

@nickvergessen
nickvergessen merged commit 9844fc3 into master Jul 3, 2019
@delete-merged-branch
delete-merged-branch Bot deleted the bugfix/1872/leave-call-and-room-when-logging-out branch July 3, 2019 09:07
@backportbot-nextcloud

Copy link
Copy Markdown

backport to stable16 in #1947

@danxuliu

danxuliu commented Jul 4, 2019

Copy link
Copy Markdown
Member

@nickvergessen The compatibility test fails with

OC_Hook - Static method of private class must not be called
App is not compliant 

@nickvergessen

Copy link
Copy Markdown
Member Author

It's compliance, not compatibility ;)

Accepted risk, we could disable the check altogether, but we can have a look at it now and then.
The other way would be to add an event dispatcher in the place the hook is triggered.

@nickvergessen

Copy link
Copy Markdown
Member Author

Nevermind, see #1951 for a fix

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

"XXX left the call" message is not shown if the user log outs while in the call

2 participants