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.
Hey @jordancde - had a chat with Alex and this is where we got to for changes in the web SDK to allow for video recordings. Should be pretty minor.
Including this PR as an example of the following changes.
Main open question is whether or not you guys use the
cloud
orraw
recording setting in DailyCoChanges required in vapi/web
web/vapi.ts
Line 108 in e1bb6e7
web/api.ts
Line 1860 in e1bb6e7
This would then be passed to the Vapi backend in the [body](
web/api.ts
Line 3663 in e1bb6e7
/call/web
Pass this flag to the DailyCall to join with the video source enabled
Changes required in vapi api
videoRecordingUrl
to the response when a request is made to the [Get Call](https://docs.vapi.ai/api-reference/calls/get-call) endpointor
recordingUrl
returns .mp4 recording from Daily.co