Voice Namespace in Realtime-API #477
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
🦋 Changeset detectedLatest commit: 9fe8d88 The changes in this PR will be included in the next version bump. This PR includes changesets to release 7 packages
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 |
edolix
force-pushed
the
eg/rt-api-voice-namespace
branch
2 times, most recently
from
cd5d70b
to
4a36870
Compare
edolix
commented
Apr 4, 2022
framini
reviewed
Apr 4, 2022
| node_id: this.nodeId, | ||
| state: 'playing', | ||
| } | ||
| this.emit(toLocalEvent('calling.playback.start'), startEvent) |
There was a problem hiding this comment.
Definitely not for this PR, but we should probably come up with an alternative for this in the future. Feels like a bit too much indirection and overhead for what we're looking after.
| }) | ||
| } | ||
|
|
||
| waitUntilConnected() { |
There was a problem hiding this comment.
from our chat today: discussed having a generic waitFor(<event>) method.
framini
force-pushed
the
eg/rt-api-voice-namespace
branch
2 times, most recently
from
da44c90
to
143b574
Compare
edolix
force-pushed
the
eg/rt-api-voice-namespace
branch
from
76ddae2
to
652eb00
Compare
* Add call answer and update Voice workers * voice playground with ENVs * remove call.state handle from playground * improve types * remove types for created/ended voice events * add changeset * type voice worker
* Voice call is not a BaseConsumer anymore * add voice play types and split workers on different files * add play method on Call * [ot] fix jest warnings and use room_session instead of room from room.subscribed event * realtime-api jest fixes * test coverage for CallPlayback * improve types and remove ts-x notations * add derived methods * changeset
* fix import * rename CallClient into VoiceClient * types * welcome AutoApplyTransformsConsumer * move createCallObject to its own emitter transform
* add types * CallRecording object * add worker voiceCallRecordWorker * implement record/recordAudio methods * update playground example with record * add changeset
* add prompt types * CallPrompt object and specs * add voiceCallPromptWorker * add prompt method to Voice Call object * improve CallPrompt public interface * update playground with prompt example * changeset * passing tests * add waitForResult in CallPrompt object
* add types for tap * CallTap obj and specs * add voiceCallTapWorker * implement tap/tapAudio methods on Voice Call * changeset * tweaks
* add types for relay connect methods/events * update toInternalDevices for sip case * add voiceCallConnectWorker * update voiceCallStateWorker to always dispatch an event and filter for state event by call_id * add connect, disconnect and waitUntilConnected on the Call and provide a playground * changeset * move call.state listener on constructor
* add util for converting a record's props to snake_case * export util from core * update `record` method to use camelCase * improve typings for toSnakeCaseProps * rename function, update prompt method * convert remaining prompt params * use camelCase for devices and add conversion for the internal util * convert digit props * add util for checking typeof * add ability to convert deeply nested arrays * fix typeof * fix condition * remove need of the util
* rename var to match other workers * split sagas into state and dial events * wip for having the dialWorker handling dial events * restore type * remove call_state filter so the worker can update the instance with all the call.state events * add ability to handle errors on dial, remove loop * remove unused event * remove handlers when the first event occurs * type array * remove unnecessary check
…496) * add new apis for handling workers * wip for moving to the new api, remove synthetic events * add option to skip caching the base instance * remove last call of attachWorkers * add dirty debugging files * fix hangup logic * add logs for callId/tag in voiceCallStateWorker * do override for "tag" in calling.call.state" events * kill the voiceCallStateWorker on ended * prettier * fix params access * remove todos * update comment in voiceCalStateWorker * add comment on voiceCallPlayWorker * remove commented import * fix casing * format comments * add changeset Co-authored-by: Edoardo Gallo <edo91.gallo@gmail.com>
* change signature for the onError * add types for sendDigits * add sendDigits method * add changeset * update error message * update types to allow passing an initialState, filter by controlid * pr feedback
* initial proposal for voice createDialer * move createDialer from VoiceClient to an util - moved tests * remove from client and update playground * change dialX with addX * changeset * add method to dialPhone and dialSip directly * update playground with examples
* add createPlaylist and types * update methods in Call and playground * expose createPlaylist from the Voice namespace instead of the Client * changeset * move tests
* add types for detect * CallDetect object * add voiceCallDetectWorker * initial implementation for voice call detect * update detect method * more detect types * on vs once * add waitForResult to Detect * update voiceCallDetectWorker * logger info to trace * add trace debug on pubSubSaga * make CallDetect work and expose Detector type * update playground with detect * derived methods for detect and snake_case vs camelCase params * add method to call contract and update playground * changeset * update detect typing
* make api as internal * move play to runWorker, fix play return type * move record to runWorker * move prompt to runWorker * move tap to runWorker * replace payload with initialState, change signature to give room for passing the params coming from the server * move detect to runWorker * move voiceCallReceiveWorker to runWorker * add ability to pass custom types for worker hooks * add proper typings to each of the onDone/onFail for all workers * rollback changes for connect worker * make generic optional for backwards compat * prettier * add changeset
* create Dialer class * move implementation to use new Dialer() * add Playlist object * update implementation to use the new Playlist constructor * changeset * fix VoicePlaylist add signature
* change on to once on CallPrompt * add waitForEnded in CallPlayback * dispatch proper event from worker to update CallPlayback * update playground * add changeset * Update packages/realtime-api/src/voice/workers/voiceCallPlayWorker.ts Co-authored-by: Francisco Ramini <framini@gmail.com> Co-authored-by: Francisco Ramini <framini@gmail.com>
* add e2e for voice namespace * update drone.yml Co-authored-by: Edoardo Gallo <edo91.gallo@gmail.com>
edolix
force-pushed
the
eg/rt-api-voice-namespace
branch
from
4c28396
to
9fe8d88
Compare
edolix
changed the title
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This branch will includes all the other branches for each voice-feature.