-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
demos: update webrtc adapter 8.0.0 #2702
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! I'll have to make some tests to make sure nothing break in the demos, before merging.
@@ -33,6 +33,6 @@ | |||
"tests" | |||
], | |||
"dependencies": { | |||
"webrtc-adapter": "7.7.0" | |||
"webrtc-adapter": "8.0.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you test this too? I remember bower having issues because the version available online wasn't available there too.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just running sorry, you're asking about bower:npm install
within the janus directory, but worked as expected (it found webrtc-adapter 8.0.0 and installed it in node_modules
).
tmatth@bellini:~/src/janus-gateway$ ./node_modules/bower/bin/bower install
bower webrtc-adapter#8.0.0 not-cached https://github.com/webrtc/adapter.git#8.0.0
bower webrtc-adapter#8.0.0 resolve https://github.com/webrtc/adapter.git#8.0.0
bower webrtc-adapter#8.0.0 download https://github.com/webrtc/adapter/archive/v8.0.0.tar.gz
bower webrtc-adapter#8.0.0 extract archive.tar.gz
bower webrtc-adapter#8.0.0 resolved https://github.com/webrtc/adapter.git#8.0.0
bower webrtc-adapter#8.0.0 install webrtc-adapter#8.0.0
webrtc-adapter#8.0.0 bower_components/webrtc-adapter
22bc7fd
to
e4ad491
Compare
Breaking changes [1]: * remove legacy edge + rtcpeerconnection-shim (see meetecho#1076) * RTCIceCandidate polyfill is now polyfilled according to spec (see meetecho#999) [1] https://github.com/webrtcHacks/adapter/releases/tag/v8.0.0
e4ad491
to
fc41bcb
Compare
Ok, we finally managed to test with Edge, which worked (at least what we tried) so I'll merge, thanks 👍 |
I should clarify that we tested with the new Chromium-based Edge, not the old one: but since I don't really care about that one, I think we can safely ignore it. |
The issue with webrtc-adapter tags not being published to cdnjs has been resolved, see:
cdnjs/packages#746
Tested locally (note: I don't have edge).