Skip to content

Unable to fetch voices for AWS Polly #1192

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

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

matteopilotto
Copy link

Keep getting the following error:
TypeError: puter.ai.listVoices is not a function

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@KernelDeimos
Copy link
Contributor

I wasn't able to reprod this in a manual test. You mentioned you already tried hard-reload without cache. Was dev-tools open you used Ctrl+Shift+R? I've found that the hotkey doesn't actually work unless devtools is open (there's no visibility on this, so that's actually a UX bug in chromium IMO). It's also possible there's something else different about your environment that's causing puter.js builds to fail.

pr-1192_1.6x.mp4

@matteopilotto
Copy link
Author

@KernelDeimos thanks for the recording.

I tested the function following your approach, step-by-step, and it actually work.
I assume the problem lies in the way I'm using it in the polly-app.
In the app I'm also calling puter.ai.txt2speech(text), but in that case the function works as expected without any error.

If you have a chance, could you also take a look at the code inside the polly-app directory?
Thanks

@KernelDeimos
Copy link
Contributor

@matteopilotto I see the problem, your script url is https://js.puter.com/v2/ - well, you won't get puter.ai.newFeature until it's deployed to js.puter.com (which in this case, it is not). In development you can use /puter.js/v2 as the URL (with no protocol or domain) to get the local version of puter.js.

@matteopilotto
Copy link
Author

Oh! I see...
Now things are working properly now.
Thanks a lot for the support Eric.
Much (much) appreciated.

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.

3 participants