Skip to content
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

Introduce intent URL param and make it change the default lobby behaviour #2828

Open
wants to merge 2 commits into
base: livekit
Choose a base branch
from

Conversation

hughns
Copy link
Member

@hughns hughns commented Nov 22, 2024

Proposal: if the user is starting a call in widget mode then we should take them straight into the call rather than the lobby.

Otherwise, the widget host will have sent the call notify (which will have rung other people) but the person who started the call not actually joined yet... which is worked around in EW/ED at the moment.

Best viewed without whitespace changes.

@hughns hughns changed the title Refactor URL parameters into table Introduce intent URL param and make it change the default lobby behaviour Nov 22, 2024
@hughns hughns added the X-Needs-Product More input needed from the Product team label Nov 22, 2024
Base automatically changed from hughns/url-params-table to livekit November 23, 2024 08:51
@hughns hughns force-pushed the hughns/url-intent-param branch from 24ea34c to 5769ead Compare November 23, 2024 09:07
@fkwp
Copy link
Contributor

fkwp commented Dec 9, 2024

sounds like a good plan

@fkwp fkwp removed the X-Needs-Product More input needed from the Product team label Dec 9, 2024
@hughns hughns force-pushed the hughns/url-intent-param branch from 5769ead to c7767e8 Compare December 9, 2024 09:38
@hughns hughns marked this pull request as ready for review December 9, 2024 09:40
@hughns hughns requested a review from a team as a code owner December 9, 2024 09:40
@hughns hughns requested a review from robintown December 9, 2024 09:40
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