-
Notifications
You must be signed in to change notification settings - Fork 0
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
[WIP] Video Call #60
base: master
Are you sure you want to change the base?
[WIP] Video Call #60
Conversation
Codecov Report
@@ Coverage Diff @@
## master #60 +/- ##
=======================================
Coverage 94.62% 94.62%
=======================================
Files 22 22
Lines 428 428
=======================================
Hits 405 405
Misses 23 23 Continue to review full report at Codecov.
|
* master: Add Heroku generated app.json
@Alir3z4 what do you think we should do about this? If we can't even get it working, I don't want this to be merged. |
No don't get this merged.
For now let it stay here until we get back to it later.
Don't need to do anything about it for now.
Let's get back on master and improve the current stable version
…On Tue, Jun 19, 2018 at 9:51 AM Amir Savand ***@***.***> wrote:
@Alir3z4 <https://github.com/Alir3z4> what do you think we should do
about this? If we can't even get it working, I don't want this to be merged.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#60 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAkFCV58_uTT4SElB-wa19yXgn5dm9A3ks5t-J93gaJpZM4Upy2v>
.
|
Fixes #59
There's https://github.com/bahmutov/ng-simple-webrtc as well