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

MSC3417: Call room room type #3417

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

SimonBrandner
Copy link
Contributor

@SimonBrandner SimonBrandner commented Sep 30, 2021

Requires #3401

Rendered

Signed-off-by: Šimon Brandner <simon.bra.ag@gmail.com>
@SimonBrandner SimonBrandner changed the title Call room room type MSC3417: Call room room type Sep 30, 2021
Signed-off-by: Šimon Brandner <simon.bra.ag@gmail.com>
@turt2live turt2live added client-server Client-Server API kind:feature MSC for not-core and not-maintenance stuff proposal A matrix spec change proposal needs-implementation This MSC does not have a qualifying implementation for the SCT to review. The MSC cannot enter FCP. labels Sep 30, 2021
Signed-off-by: Šimon Brandner <simon.bra.ag@gmail.com>
Copy link
Member

@turt2live turt2live left a comment

Choose a reason for hiding this comment

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

generally seems fine, and very straightforward - thanks for writing it up!

proposals/3417-call-room-type.md Outdated Show resolved Hide resolved
proposals/3417-call-room-type.md Outdated Show resolved Hide resolved
SimonBrandner and others added 2 commits March 19, 2022 08:03
Co-authored-by: Travis Ralston <travpc@gmail.com>
Signed-off-by: Šimon Brandner <simon.bra.ag@gmail.com>
Comment on lines +53 to +56
event, it is mutable which we'd like to avoid in the case of call rooms. Clients
supporting call rooms will present the chat in those rooms as secondary or not
present it at all, so it is beneficial for this to be immutable so that the chat
history isn't "lost" when changing the room type.

Choose a reason for hiding this comment

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

I argue that using a separate room type has a different meaning entirely, and I give m.space as an example of this: it's very obvious that an m.space room is not a regular chat room and should not be used for chat messages (though it technically can). Should that assumption apply to all explicitly typed rooms in general? Should a chat client that does not recognize a room type hide that room from its room list? If so, doesn't that mean the chat of a call room becomes invisible to those clients, even though a user might reasonably expect to be able to see the chat of a call room even if a client doesn't support calls?

This concern with room types might also apply to MSC3815 (Third Room Worlds), where it might make sense for a client user to be able to see the text chat of a room with an ongoing World, even if the client doesn't directly support Third World.

I believe the m.room call intent already implies that any regular room can become a call room in the sense of room mutability, and as such there is no special distinction between a regular room and a call room for the purposes of text chat. So I'm not sure why a client would hide the chat history of such a room if a room call is started. Unless the concern is about UX confusion where the chat is folded away.

I tripped on this issue while messing around with Element sending call state events with the room intent; Element (and by extension matrix-js-sdk) does not seem to recognize Rooms as having ongoing m.room calls yet unless the room itself has the UnstableCall (org.matrix.msc3417.call) type. So in the presence of this MSC and MSC3401, it is already unclear what a conforming client should do with m.room intent calls.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
client-server Client-Server API kind:feature MSC for not-core and not-maintenance stuff needs-implementation This MSC does not have a qualifying implementation for the SCT to review. The MSC cannot enter FCP. proposal A matrix spec change proposal voip
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants