Skip to content

Update AccessToken.ts#509

Closed
yaniv-peretz wants to merge 1 commit into
livekit:mainfrom
yaniv-peretz:patch-1
Closed

Update AccessToken.ts#509
yaniv-peretz wants to merge 1 commit into
livekit:mainfrom
yaniv-peretz:patch-1

Conversation

@yaniv-peretz

Copy link
Copy Markdown

Not sure, but it's it required to allow users connect to a room (mine failed without it)

@changeset-bot

changeset-bot Bot commented Jul 19, 2025

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 2b3fd39

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@CLAassistant

CLAassistant commented Jul 19, 2025

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@davidzhao

Copy link
Copy Markdown
Member

optional is actually the right thing here. for room connect tokens, it's required, but for other types of tokens it isn't needed

@davidzhao davidzhao closed this Aug 8, 2025
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.

3 participants