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

Make the user pass in a WebRTC implementation (for #7) #12

Closed
wants to merge 1 commit into from

Conversation

paulkernfeld
Copy link

Hopefully this change isn't too invasive. This seemed like the most straightforward way to do it.

Also a quick note that this is a breaking change.

@mappum
Copy link
Owner

mappum commented Feb 5, 2016

Hey @paulkernfeld, thanks for contributing! Although I should let you know that a lot of the code inside this repo is already being moved into smaller repos, see the roadmap in the README for links.

@paulkernfeld
Copy link
Author

Ah, okay, now I see that this is approach is already implemented in peerGroup in bitcoin-net for example, so that will make this PR obsolete soon. Do you have an ETA for finishing the refactor?

@mappum
Copy link
Owner

mappum commented Feb 5, 2016

Do you have an ETA for finishing the refactor?

I'm looking to get it done ASAP, but I'm bad at time estimates. Maybe 3 or 4 days of work. #13

@paulkernfeld
Copy link
Author

Nice! That's super fast, so it's probably not worth it to merge this PR.

@mappum
Copy link
Owner

mappum commented Feb 28, 2016

Closing because this feature is in bitcoin-net.

@mappum mappum closed this Feb 28, 2016
@paulkernfeld
Copy link
Author

Awesome!

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