-
Notifications
You must be signed in to change notification settings - Fork 0
Realtimekit/meeting object page #11
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
base: production
Are you sure you want to change the base?
Conversation
|
This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:
|
|
|
||
| <RTKCodeSnippet id="web-react"> | ||
|
|
||
| The [`meeting.self`](https://docs.realtime.cloudflare.com/web-core/reference/RTKSelf) represents the local user (you) in the meeting. It provides properties and methods to control your own audio, video, and screen sharing. |
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.
This link needs to be a relative path to the RTKSelf page in developers.cloudflare.com
|
|
||
| <RTKCodeSnippet id="web-web-components"> | ||
|
|
||
| The [`meeting.self`](https://docs.realtime.cloudflare.com/web-core/reference/RTKSelf) represents the local user (you) in the meeting. It provides properties and methods to control your own audio, video, and screen sharing. |
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.
Same thing here as well
|
|
||
| <RTKCodeSnippet id="mobile-react-native"> | ||
|
|
||
| The [`meeting.self`](https://docs.realtime.cloudflare.com/mobile-core/reference/RTKSelf) represents the local user (you) in the meeting. It provides properties and methods to control your own audio, video, and screen sharing. |
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.
Same here
| // All participants (includes both joined and not joined) | ||
| val all: List<RtkParticipant> = meeting.participants.all |
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.
This was removed as the backing list never gets populated.
| // Active speaker | ||
| val activeSpeaker: RtkRemoteParticipant? = meeting.participants.activeSpeaker |
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.
Yeet
| firstParticipant?.id // Participant ID (aka peerId) | ||
| firstParticipant?.userId // User ID | ||
| firstParticipant?.name // Display name | ||
| firstParticipant?.picture // Participant picture (if any) | ||
| firstParticipant?.customParticipantId // Custom participant ID | ||
| firstParticipant?.audioEnabled // Audio state | ||
| firstParticipant?.videoEnabled // Video state | ||
| firstParticipant?.screenShareEnabled // Screen share state | ||
| firstParticipant?.isPinned // Pin state | ||
| firstParticipant?.isHost // Host state | ||
| firstParticipant?.presetName // Preset name | ||
| firstParticipant?.stageStatus // Stage status | ||
| firstParticipant?.flags // Participant flags (recorder, hiddenParticipant, webinarHiddenParticipant) |
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.
These are documented later as well, we should delete this.
| let activeSpeaker: RtkRemoteParticipant? = meeting.participants.activeSpeaker | ||
|
|
||
| // All participants (includes both joined and not joined) | ||
| let all: [RtkParticipant] = meeting.participants.all |
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.
Yeet
| firstParticipant?.id // Participant ID (aka peerId) | ||
| firstParticipant?.userId // User ID | ||
| firstParticipant?.name // Display name | ||
| firstParticipant?.picture // Participant picture (if any) | ||
| firstParticipant?.customParticipantId // Custom participant ID | ||
| firstParticipant?.audioEnabled // Audio state | ||
| firstParticipant?.videoEnabled // Video state | ||
| firstParticipant?.screenShareEnabled // Screen share state | ||
| firstParticipant?.isPinned // Pin state | ||
| firstParticipant?.isHost // Host state | ||
| firstParticipant?.presetName // Preset name | ||
| firstParticipant?.stageStatus // Stage status | ||
| firstParticipant?.flags // Participant flags (recorder, hiddenParticipant, webinarHiddenParticipant) |
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.
Same as above
| firstParticipant?.id; // Participant ID (aka peerId) | ||
| firstParticipant?.userId; // User ID | ||
| firstParticipant?.name; // Display name | ||
| firstParticipant?.picture; // Participant picture (if any) | ||
| firstParticipant?.customParticipantId; // Custom participant ID | ||
| firstParticipant?.audioEnabled; // Audio state | ||
| firstParticipant?.videoEnabled; // Video state | ||
| firstParticipant?.screenShareEnabled; // Screen share state | ||
| firstParticipant?.isPinned; // Pin state | ||
| firstParticipant?.isHost; // Host state | ||
| firstParticipant?.presetName; // Preset name | ||
| firstParticipant?.stageStatus; // Stage status | ||
| firstParticipant?.flags; // Participant flags (recorder, hiddenParticipant, webinarHiddenParticipant) |
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.
Yeet
Creating a draft PR for review comments because commit comments will notify everyone watching the repo