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

Resolve the .ended() promise in case of call failure #732

Merged
merged 13 commits into from
Feb 16, 2023
Merged

Conversation

iAmmar7
Copy link
Collaborator

@iAmmar7 iAmmar7 commented Jan 30, 2023

Based on this issue.

  • Handle the .ended() promise for Playback function

  • Handle the .ended() promise for Prompt function

  • Handle the .ended() promise for Recording function

  • Handle the .ended() promise for Detect function

  • Handle the .ended() promise for Collect function

  • Handle the .ended() promise for Tap function

  • e2e test cases for Playback function

  • e2e test cases for Prompt function

  • e2e test cases for Detect function

  • e2e test cases for Tap function

  • e2e test cases for Recording function

@changeset-bot
Copy link

changeset-bot bot commented Jan 30, 2023

🦋 Changeset detected

Latest commit: cbf85e9

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 6 packages
Name Type
@signalwire/realtime-api Minor
@signalwire/core Minor
@signalwire/js Patch
@signalwire/react-native Patch
@signalwire/web-api Patch
@signalwire/webrtc Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@iAmmar7 iAmmar7 changed the title WIP: Resolve the ended promise in case of call failure WIP: Resolve the .ended() promise in case of call failure Jan 30, 2023
@iAmmar7 iAmmar7 requested a review from edolix January 31, 2023 14:08
Copy link
Contributor

@edolix edolix left a comment

Choose a reason for hiding this comment

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

Looks great! Just a small comment about the sleep function.

.changeset/seven-beers-occur.md Outdated Show resolved Hide resolved
internal/e2e-realtime-api/src/voiceDetect.test.ts Outdated Show resolved Hide resolved
internal/e2e-realtime-api/src/voiceRecording.test.ts Outdated Show resolved Hide resolved
internal/e2e-realtime-api/src/voiceTap.test.ts Outdated Show resolved Hide resolved
@edolix edolix changed the title WIP: Resolve the .ended() promise in case of call failure esolve the .ended() promise in case of call failure Feb 6, 2023
@edolix edolix changed the title esolve the .ended() promise in case of call failure Resolve the .ended() promise in case of call failure Feb 6, 2023
@edolix
Copy link
Contributor

edolix commented Feb 15, 2023

Rebased

@edolix edolix merged commit 9ad158b into main Feb 16, 2023
@edolix edolix deleted the aa/ended-failure branch February 16, 2023 09:09
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.

2 participants