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

Update from deprecated API: AudioContext.createScriptProcessor() #205

Closed
trevordilley opened this issue May 14, 2020 · 3 comments · Fixed by #212
Closed

Update from deprecated API: AudioContext.createScriptProcessor() #205

trevordilley opened this issue May 14, 2020 · 3 comments · Fixed by #212

Comments

@trevordilley
Copy link

Hi Chris, we have a client who would like switch over to your recorder as their main recording tool. The one snag is they're uncomfortable that the recorder uses this:
https://developer.mozilla.org/en-US/docs/Web/API/BaseAudioContext/createScriptProcessor

Which appears to be deprecated? Currently they're using the Twilio but would like to use something simpler (they just need to record a single voice recording and upload it to their servers, so Twilio is kinda overkill)

We know this would be a non-trivial effort and would be happy to follow up and discussing bidding it out. You can contact me directly at trevor@twentyideas.com if you want to talk more!

@trevordilley
Copy link
Author

Specifically here:

this.scriptProcessorNode = this.audioContext.createScriptProcessor( this.config.bufferLength, this.config.numberOfChannels, this.config.numberOfChannels );

@chris-rudmin
Copy link
Owner

chris-rudmin commented May 15, 2020

@trevordilley deprecating createScriptProcessor() sounds reasonable. Just FYI for your client, that making this change will require dropping support for iOS and Safari on mac.

@weepy
Copy link

weepy commented Jun 5, 2020

"dropping support for iOS and Safari " seems a shame since they are the main reasons this exists!

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 a pull request may close this issue.

3 participants