Skip to content

[FCE-2938] Re-enable simulcast variant switching#496

Merged
czerwiukk merged 15 commits intomainfrom
sgfn/simulcast
Apr 1, 2026
Merged

[FCE-2938] Re-enable simulcast variant switching#496
czerwiukk merged 15 commits intomainfrom
sgfn/simulcast

Conversation

@sgfn
Copy link
Copy Markdown
Contributor

@sgfn sgfn commented Mar 16, 2026

Description

  • Re-enables simulcast in webrtc-client
  • Adds a new useSetTargetTrackEncoding hook in react-client
  • Propagates the existing per-track encodingChanged event up to the client-level MessageEvents in ts-client
  • Updates the minimal-react example to enable simulcast and render variant selection buttons

Motivation and Context

FCE-2938
Simulcast was deliberately disabled pending manual track selection support.

Documentation impact

  • Documentation update required
  • Documentation updated in another PR
  • No documentation update required

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to
    not work as expected)

@sgfn sgfn requested review from Karolk99 and czerwiukk March 16, 2026 13:50
@linear
Copy link
Copy Markdown

linear bot commented Mar 16, 2026

Copy link
Copy Markdown
Contributor

@Karolk99 Karolk99 left a comment

Choose a reason for hiding this comment

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

As discussed in the office, we don't need this event at the moment. Working example and setVideoQuality function instead of setTrackTargetVariant is enough for now. Also, you can add a test for switching layers and simulcast as another integration test.

@czerwiukk
Copy link
Copy Markdown
Member

tests failing due to staging outage

@czerwiukk czerwiukk merged commit 91e5352 into main Apr 1, 2026
2 of 5 checks passed
@czerwiukk czerwiukk deleted the sgfn/simulcast branch April 1, 2026 12:23
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