-
Notifications
You must be signed in to change notification settings - Fork 2
Batch API: Add support for fetch_data to provide an audio URL #32
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
Batch API: Add support for fetch_data to provide an audio URL #32
Conversation
|
@dumitrugutu @anjz Can you review and hopefully publish a release with this soon? That would help me greatly! Thanks :) |
As per the API, it is possible to pass an audio url via fetch_data instead of an actual audio file. This implements that functionality as per API spec.
|
Hi @TheCrazyLex, thanks for your contribution :) We'll review it shortly. |
anjz
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your contribution. It looks great. I've just added a few suggestions.
Co-authored-by: Anartz Nuin <anartzn@gmail.com> Co-authored-by: Dumitru Gutu <dumitrugutu@users.noreply.github.com>
Co-authored-by: Anartz Nuin <anartzn@gmail.com>
|
@dumitrugutu @anjz Implemented your review comments and verified functionality with these changes on my side. |
|
🚀 |
As per the API, it is possible to pass an audio url via fetch_data instead of an actual audio file.
This implements that functionality as per API spec.
I tested this feature, and it fully works as expected with this code :)