Skip to content

Conversation

@xianshijing-lk
Copy link
Collaborator

This PR implements the remaining room events
and separate the room_event_types.h and room_delegate.h

It also adds the comments / docs to the room_delegates.h and room_event_types.h

Copy link

@cloudwebrtc cloudwebrtc left a comment

Choose a reason for hiding this comment

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

lgtm!

Copy link

@ladvoc ladvoc left a comment

Choose a reason for hiding this comment

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

LGTM, just a few notes.

/**
* A chat message associated with the room.
*/
struct ChatMessageData {
Copy link

@ladvoc ladvoc Dec 10, 2025

Choose a reason for hiding this comment

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

note: Chat messages are deprecated in favor of data streams. Not sure if we want to expose in new code.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Good find, I removed all the code related to ChatMessageData, as it is a deprecated feature, we shouldn't ship it.

return out;
}

proto::TranscriptionReceived toProto(const Transcription &in) {
Copy link

Choose a reason for hiding this comment

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

note: Same with transcription.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

how about TranscriptionSegment ?

@xianshijing-lk xianshijing-lk merged commit 6457ca9 into main Dec 11, 2025
5 checks passed
@xianshijing-lk xianshijing-lk deleted the sxian/CLT-2342/implement_the_remaining_room_events branch December 11, 2025 22:56
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.

4 participants