-
Notifications
You must be signed in to change notification settings - Fork 11k
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
Can't access online demo: ReferenceError: mozRTCSessionDescription is not defined
#265
Comments
Which browser and version are you using? |
Mozilla/5.0 (X11; Fedora; Linux x86_64; rv:38.0) Gecko/20100101 Firefox/38.0 |
We may want to detect for webrtc support within our package code before executing any line that can fail. Then ensure the "Video Button" does not show when there is no support. It will avoid having to troubleshoot multiple variations of browser + OS combinations (similar to this issue) going forward. In this field tested Apache Licensed code - see the isWebRtcCompatible() method for a possible implementation: |
I thought that we could do this using just the adapter.js we copied from https://github.com/webrtc/adapter There are other alternatives: https://github.com/Temasys/AdapterJS Maybe we need to spend more time to investigate the best solution. Thanks for the extra info @Sing-Li |
I think the 'webrtc compatibility charts' all over the web are misleading. Just because it says Firefox 38 supports webrtc - it does not mean ALL firefox-38 + OS combinations support webrtc. This is why run-time, browser capabilities-based detection of webrtc support remains very important in production systems. |
I am working on it. I've tried to use AdapterJS, looks good, but do not worked for me easily, so I changed the old adapter to a much more simple code and now webrtc is working in FF in OS X. The next step is create this verifications if the browser support webrtc and then work with some solution to make webrtc work in IE and Safari. |
@guarilha Can you test again? |
it's working now @rodrigok 👍 |
:D |
…smarti-adapter-livechat Fix Smarti-Adapter for livechats
The text was updated successfully, but these errors were encountered: