Skip to content

Allow multiple MatrixRTC sessions per room #3451

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

Merged
merged 3 commits into from
Aug 19, 2025

Conversation

fkwp
Copy link
Contributor

@fkwp fkwp commented Aug 13, 2025

@fkwp fkwp changed the title bump js-sdk to Allow multiple rtc sessions per room (with different s… bump js-sdk to Allow multiple MatrixRTC sessions per room Aug 14, 2025
yarn.lock Outdated
@@ -7537,7 +7537,7 @@ __metadata:
livekit-client: "npm:^2.13.0"
lodash-es: "npm:^4.17.21"
loglevel: "npm:^1.9.1"
matrix-js-sdk: "github:matrix-org/matrix-js-sdk#head=develop"
matrix-js-sdk: "github:matrix-org/matrix-js-sdk#7a93d2bf150e9f84bc6397b18cf5f92cc0c78b8b"
Copy link
Member

Choose a reason for hiding this comment

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

Let's make sure to reset this to head=develop once the matrix-js-sdk PR is merged

Copy link
Contributor Author

Choose a reason for hiding this comment

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

makes sense -- we can do that once matrix-org/matrix-js-sdk#4945 is merged

Copy link
Contributor

@toger5 toger5 Aug 19, 2025

Choose a reason for hiding this comment

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

matrix-org/matrix-js-sdk#4945 is merged and I updated the package.json

@fkwp fkwp changed the title bump js-sdk to Allow multiple MatrixRTC sessions per room Allow multiple MatrixRTC sessions per room Aug 18, 2025
@toger5 toger5 removed the X-Blocked Cannot be merged due to external dependencies label Aug 19, 2025
@toger5
Copy link
Contributor

toger5 commented Aug 19, 2025

I took over the last js-sdk bump for this PR: 8a71883

@toger5 toger5 marked this pull request as ready for review August 19, 2025 12:59
@toger5 toger5 requested a review from a team as a code owner August 19, 2025 12:59
@toger5 toger5 requested a review from AndrewFerr August 19, 2025 12:59
@fkwp fkwp added the PR-Feature Release note category. A PR that introduces a new user facing feature. label Aug 19, 2025
@fkwp fkwp merged commit a0d7780 into livekit Aug 19, 2025
32 of 34 checks passed
@toger5 toger5 deleted the fkwp/experimental/test-new-state_key_format branch August 19, 2025 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR-Feature Release note category. A PR that introduces a new user facing feature.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants