Skip to content
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

feat: expose icecandidateerror event #82

Merged

Conversation

k-wasniowski
Copy link
Contributor

SPARK-536950

This PR introduces forwarding of IceCandidateError event. We would like to be able to use it in JS-SDK to have a metrics about most common ice gathering issues.

With typescript update to version > 5.0, cast would not be needed.

bbaldino
bbaldino previously approved these changes Jun 27, 2024
Copy link
Collaborator

@bbaldino bbaldino left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, but maybe we want to wait for the typescript upgrade?

@k-wasniowski
Copy link
Contributor Author

lgtm, but maybe we want to wait for the typescript upgrade?

Agree, created this PR just in case if we wouldn't want to upgrade typescript.

brycetham
brycetham previously approved these changes Jun 27, 2024
@k-wasniowski k-wasniowski dismissed stale reviews from brycetham and bbaldino via 67ecc0d July 10, 2024 17:26
@k-wasniowski k-wasniowski force-pushed the spark-536950-add-metrics-about-icecandidate-error-codes branch from caf0b80 to 67ecc0d Compare July 10, 2024 17:26
@k-wasniowski k-wasniowski merged commit 29669bd into main Jul 11, 2024
1 check passed
@k-wasniowski k-wasniowski deleted the spark-536950-add-metrics-about-icecandidate-error-codes branch July 11, 2024 07:43
bbaldino pushed a commit that referenced this pull request Jul 11, 2024
# [2.10.0](v2.9.0...v2.10.0) (2024-07-11)

### Features

* expose icecandidateerror event ([#82](#82)) ([29669bd](29669bd))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants