Skip to content

[VID-1267] Pinned flag on participant joined#1772

Merged
brunoribc merged 1 commit intomainfrom
send-pinned-on-joined-event
Apr 7, 2026
Merged

[VID-1267] Pinned flag on participant joined#1772
brunoribc merged 1 commit intomainfrom
send-pinned-on-joined-event

Conversation

@brunoribc
Copy link
Copy Markdown
Contributor

Adding an isPinned flag for participantJoined

@brunoribc brunoribc changed the title [VID-1267] Pinned flag on pariticpant joined [VID-1267] Pinned flag on participant joined Apr 7, 2026
@brunoribc brunoribc merged commit 0a7d47e into main Apr 7, 2026
2 checks passed
@brunoribc brunoribc deleted the send-pinned-on-joined-event branch April 7, 2026 13:46
@github-actions github-actions bot mentioned this pull request Apr 7, 2026
oliverlaz added a commit to GetStream/stream-video-js that referenced this pull request Apr 9, 2026
### 💡 Overview

Support server-side pinning when a participant joins a call. When the
SFU signals `isPinned: true` on a `ParticipantJoined` event, the
participant is automatically pinned with `isLocalPin: false`.

### 📝 Implementation notes

- Updated `watchParticipantJoined` to apply a server-side pin (`{
isLocalPin: false, pinnedAt: Date.now() }`) when `e.isPinned` is truthy
- Added two unit tests: one verifying the pin is set when `isPinned:
true`, another verifying no pin is set when `isPinned: false`

Ref: GetStream/protocol#1772

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

## Summary by CodeRabbit

* **New Features**
* Participants now include pin metadata when joining while pinned,
tracking pin status and timestamp information.

* **Tests**
* Added comprehensive unit tests to validate participant pin behavior
during join events, ensuring correct handling for both pinned and
unpinned participants.

<!-- end of auto-generated comment: release notes by coderabbit.ai -->
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.

2 participants