Get rid of "[audio|video][On|Off]" datachannel events
Open
@Ivansss is already working on this.
Since Aug 31, 2017.
feature: WebRTC 🚡
standardisation
technical debt
- Dominant language
- PHP
- Stars
- 2.2k
- Forks
- 587
- Avg merge
- 18h 27m
- Merged PRs (30d)
- 333
Description
As just discussed offline with @Ivansss : currently the UI sends videoOn/videoOff and audioOn/audioOff events through the datachannels to all connected clients to notify about what is available and what is currently sending.
This is not necessary as you can get the information on media types from the offer/answer, and get the state (paused/running) from the streams. In order to remove duplicate functionality and simplify the code, these events should be removed.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.