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

feat: RPC. #682

Merged
merged 20 commits into from
Feb 19, 2025
Merged

feat: RPC. #682

merged 20 commits into from
Feb 19, 2025

Conversation

cloudwebrtc
Copy link
Contributor

@cloudwebrtc cloudwebrtc commented Jan 12, 2025

@bcherry
Copy link
Contributor

bcherry commented Feb 4, 2025

@cloudwebrtc see in linear, we also have decided to move the method handlers to room so they can be registered before or after connect. since this hasn't merged yet, no need to maintain deprecated versions on local participant

@praveen-nuna
Copy link

@bcherry Do you know when RPC be officially supported ?

@bcherry
Copy link
Contributor

bcherry commented Feb 7, 2025

@praveen-nuna we're actively working on it, should land within a week or two

@cloudwebrtc cloudwebrtc marked this pull request as ready for review February 11, 2025 01:45
lib/src/core/room.dart Outdated Show resolved Hide resolved
lib/src/core/room.dart Outdated Show resolved Hide resolved
Copy link
Contributor

@bcherry bcherry left a comment

Choose a reason for hiding this comment

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

this is great

lib/src/types/rpc.dart Outdated Show resolved Hide resolved
lib/src/types/rpc.dart Outdated Show resolved Hide resolved
lib/src/core/room.dart Outdated Show resolved Hide resolved
@cloudwebrtc cloudwebrtc requested a review from bcherry February 12, 2025 11:54
@cloudwebrtc cloudwebrtc merged commit 0405f5b into main Feb 19, 2025
8 checks passed
@cloudwebrtc cloudwebrtc deleted the duan/feat-rpc branch February 19, 2025 02: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.

4 participants