Skip to content

Make it possible to use the promise-based API#91

Open
getkey wants to merge 3 commits into
mappum:masterfrom
getkey:promise-api
Open

Make it possible to use the promise-based API#91
getkey wants to merge 3 commits into
mappum:masterfrom
getkey:promise-api

Conversation

@getkey

@getkey getkey commented Jan 13, 2017

Copy link
Copy Markdown

No description provided.

@getkey

getkey commented Jan 13, 2017

Copy link
Copy Markdown
Author

Actually that doesn't work, sorry. I'm working on a fix.

@getkey

getkey commented Jan 25, 2017

Copy link
Copy Markdown
Author

@mappum Thought on this PR?

It would be nice to support Promises since MDN says the callback API is deprecated.

@mappum

mappum commented Jan 25, 2017

Copy link
Copy Markdown
Owner

@getkey Sorry I missed this. Thanks for making this PR, keeping the API up-to-spec is important. I'll check this out/merge it later this week when I get time.

@getkey

getkey commented Jan 25, 2017 via email

Copy link
Copy Markdown
Author

@getkey

getkey commented Feb 7, 2017

Copy link
Copy Markdown
Author

@mappum Hey, sorry to bug you again but have you had time to check it out? (and PR #92, too)

@getkey

getkey commented Feb 26, 2017

Copy link
Copy Markdown
Author

I published the NPM package electron-webrtc-patched because I need the changes in this PR.
I will take it down if my PRs are accepted.

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