Fix multi-way camera / mic / screen sharing

DemocraseeClub / website

Tools and resources to turn ideas and intentions into action

4 stars 0 forks source link

Fix multi-way camera / mic / screen sharing #2

Closed eliataylor closed 3 years ago

eliataylor commented 3 years ago

go to https://democraseeclub.web.app/rally/building-democrasee/meeting/2020-11-14
Select the down arrow next to "Broadcast"
Enable your Cam/Mic/Screen in the popup
To close the popup, click outside of it
Click "Broadcast" and copy the room ID code displayed
In a new Browser or on another computer, go to https://democraseeclub.web.app/rally/building-democrasee/meeting/2020-11-14
input the code into "Enter Room ID"
click Connect

ACTUAL RESULTS

the host does not get to see the viewing party's broadcast when shared

DESIRED RESULTS

each party can view the other person's screen (if enabled)
hosts can see how many people are viewing their own broadcasts

The related files are:

https://github.com/eliataylor/clock-agendas/blob/master/src/components/Room.js https://github.com/eliataylor/clock-agendas/blob/master/src/components/VideoElement.js https://github.com/eliataylor/clock-agendas/blob/master/src/components/RemoteVideo.js

eliataylor commented 3 years ago

Que bueno @Branyer! Todos los cambios aparecen el camino correcto! Sin embargo, el huespede no puede ver los vistos de otros mirando.

eliataylor commented 3 years ago

(y pienso que https esta obligatorio para Firebase / RTCPeerConnection)

asi debemos dejar: HTTPS=true react-scripts start

Branyer commented 3 years ago

Está bien, seguiré buscando la forma de arreglar el problema.

Branyer commented 3 years ago

Hola Eli, acabo de subir cambios respecto a este issue.

eliataylor commented 3 years ago

@Branyer, yo probé y "Viewers #" funciona, pero todovia el anfitrión no puede ver el huesped

Branyer commented 3 years ago

Ya he arreglado lo del anfitrion y el huesped, pruebalo y me comentas. Aun falta algunos detalles, como que si no tiene un broadcast, cuando empiece uno, de inmediato se empiece a transmitir a la otra persona.

eliataylor commented 3 years ago

Apparently #12 is necessary to get this working 100%

eliataylor commented 3 years ago

@Branyer , yo hize #12 and tengo una nueva rama rtc-refactor con muchos cambios para este Issue.

Yo mantengo la cuenta de vistos correcto, pero no esta compartiendo el Stream. Puedes buscar si tu ves el problema?

Branyer commented 3 years ago

voy a revisar

eliataylor commented 3 years ago

@Branyer , ves algo?

Branyer commented 3 years ago

@eliataylor sí. estuve revisando y el creo que el problema es que en el Room no estas utilizando el component RemoteVideo para renderizar los videos de los viewers ni de los que se entra usando codigo, mañana me pondré a resolver este problema

Branyer commented 3 years ago

@eliataylor ya he subido un cambio sobre esto en rtc-refactor, tengo una pregunta, en este momento la funcionalidad de ver el broadcast de los viewers no funciona, me pregunto si esa funcionalidad se eliminó de la app o tengo que arreglarla para que vuelva a funcionar?

eliataylor commented 3 years ago

@Branyer , tu codigo esta en master todovia. El problema es el anfitrion del cuarto no puede ver los huespedes.

eliataylor commented 3 years ago

sigue en #39