-
Notifications
You must be signed in to change notification settings - Fork 68
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
[NEW][WIP] Add Jitsi component #100
Conversation
This pull request introduces 2 alerts when merging 4e0ff01 into 0930180 - view on LGTM.com new alerts:
|
Hi @Dnouv Thanks for the PR. A lot of this work has actually been done, as part of the virtual conference coming up - in the "Greenroom". Please see if you can "tune this component" to the following:
Adding the two above will allow us to conduct the Alumni Summit with RC4Community. |
This pull request introduces 1 alert and fixes 1 when merging a078def into 637ede7 - view on LGTM.com new alerts:
fixed alerts:
|
This pull request fixes 3 alerts when merging 6cb0fb3 into 54d6bbd - view on LGTM.com fixed alerts:
|
This pull request fixes 5 alerts when merging 0334966 into be23250 - view on LGTM.com fixed alerts:
|
This pull request fixes 5 alerts when merging 17ee690 into 7d80e8f - view on LGTM.com fixed alerts:
|
This pull request fixes 5 alerts when merging 8cbdd50 into 8651407 - view on LGTM.com fixed alerts:
|
This pull request fixes 5 alerts when merging f95f259 into 8651407 - view on LGTM.com fixed alerts:
|
This PR adds a new Jitsi component using Jitsi's React SDK
Here is a screenshot of the demo page,
Thank you!