search

elixir-webrtc / ex_webrtc

An Elixir implementation of the W3C WebRTC API
Apache License 2.0
269 stars 8 forks source link

Update bidirectional connection example in the transceiver guide #79

Closed mickel8 closed 4 months ago

mickel8 commented 4 months ago

The track notification may be triggered both when setting a remote offer and a remote answer. I've updated the example to explicitly demonstrate this case.

codecov[bot] commented 4 months ago

Codecov Report

Merging #79 (25bba4f) into master (8328e84) will not change coverage. The diff coverage is n/a.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #79 +/- ## ======================================= Coverage 88.96% 88.96% ======================================= Files 31 31 Lines 1387 1387 ======================================= Hits 1234 1234 Misses 153 153 ``` ------ [Continue to review full report in Codecov by Sentry](https://app.codecov.io/gh/elixir-webrtc/ex_webrtc/pull/79?src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=elixir-webrtc). > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=elixir-webrtc) > `Δ = absolute (impact)`, `ø = not affected`, `? = missing data` > Powered by [Codecov](https://app.codecov.io/gh/elixir-webrtc/ex_webrtc/pull/79?src=pr&el=footer&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=elixir-webrtc). Last update [8328e84...25bba4f](https://app.codecov.io/gh/elixir-webrtc/ex_webrtc/pull/79?src=pr&el=lastupdated&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=elixir-webrtc). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=elixir-webrtc).