-
Notifications
You must be signed in to change notification settings - Fork 10
Sxian/clt 2342/implement the remaining room events #14
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
Sxian/clt 2342/implement the remaining room events #14
Conversation
…h, and fix the comments
cloudwebrtc
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm!
ladvoc
left a comment
There was a problem hiding this 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.
include/livekit/room_event_types.h
Outdated
| /** | ||
| * A chat message associated with the room. | ||
| */ | ||
| struct ChatMessageData { |
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
note: Same with transcription.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
how about TranscriptionSegment ?
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