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

Bug #11441 #1075

Closed
wants to merge 1 commit into from
Closed

Bug #11441 #1075

wants to merge 1 commit into from

Conversation

mmoqui
Copy link
Member

@mmoqui mmoqui commented Apr 17, 2020

Fix the bug by overriding the WebRTC handler of the chat window init event.

Adds also a listener of events about room deletion so that the room is
automatically removed both from the user bookmarks and from his roster.

Now, when the owner of a room invites a user to join his room, the user becomes
a member of the room (in XMPP, this affiliation works only if it is done by the
owners or the admins of a room). With this information, it is now possible to
get the count of room members.

When a room is created, some properties are now explicitly set: the room is
hidden (not a public one) and it is not allowed for anyone to invite another
user.

Fix the bug by overriding the WebRTC handler of the chat window init event.

Adds also a listener of events about room deletion so that the room is
automatically removed both from the user bookmarks and from his roster.

Now, when the owner of a room invte a user to join his room, the user becomes
a member of the room (in XMPP, this affiliation works only if it is done by the
owners or the admins of a room). With this information, it is now possible to
get the count of room members.

When a room is created, some properties are now explicitly set: the room is
hidden (not a public one) and it is not allowed for anyone to invite another
user.
@sonarcloud
Copy link

sonarcloud bot commented Apr 17, 2020

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@SilverYoCha
Copy link
Member

Merged.
Closed manually because of manual git operations performed during integration.

@mmoqui mmoqui deleted the bug-11441 branch May 28, 2020 07:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants