Skip to content

fix(prejoin): Handles different ways of disabling prejoin.#16897

Merged
damencho merged 2 commits intomasterfrom
fix-disable-gum
Feb 3, 2026
Merged

fix(prejoin): Handles different ways of disabling prejoin.#16897
damencho merged 2 commits intomasterfrom
fix-disable-gum

Conversation

@damencho
Copy link
Member

@damencho damencho commented Feb 3, 2026

Now handles using config.prejoinConfig.enabled=false and `config.prejoinConfig={enabled:false}.

Fixes #16892.

@damencho damencho force-pushed the fix-disable-gum branch 4 times, most recently from 546773a to ba2b3bc Compare February 3, 2026 17:00
});
});

async function checkEveryoneMuted({ p1, p2 }: { p1: Participant; p2: Participant; }) {
Copy link
Member

Choose a reason for hiding this comment

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

why the asymmetry -- p2 checks local and remote, p1 checks only remote?

Copy link
Member Author

Choose a reason for hiding this comment

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

Fixed.

Now handles using `config.prejoinConfig.enabled=false` and `config.prejoinConfig={enabled:false}.

Fixes #16892.
@damencho
Copy link
Member Author

damencho commented Feb 3, 2026

jenkins test this please

@damencho damencho force-pushed the fix-disable-gum branch 7 times, most recently from 3c69b3c to bce853d Compare February 3, 2026 22:59
@damencho damencho merged commit 796a7ef into master Feb 3, 2026
10 checks passed
@damencho damencho deleted the fix-disable-gum branch February 3, 2026 23:58
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.

Unreachable condition in setConfigFromURLParams due to nested prejoinConfig.enabled

3 participants