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

Add support for language availability check and download #9

Merged
merged 1 commit into from Jul 4, 2017

Conversation

yjlim5
Copy link
Contributor

@yjlim5 yjlim5 commented Jun 30, 2017

Currently, the speak function fails silently if the requested language is not supported on the device. To address this issue, this PR adds:

  1. Error message from speak that requested language is not available
  2. checkLanguageAvailability(language) for checking language support
  3. downloadTTSVoice() for TTS Voice download page

Together these changes will allow developers to easily detect and handle error cases where the language support is unavailable on Android device.

@mihirsoni
Copy link
Owner

@yjlim5 Thank you so much for the PR, I will review and update it.

@yjlim5
Copy link
Contributor Author

yjlim5 commented Jul 4, 2017

@mihirsoni thank you for providing this library and reviewing my PR! We have been using this library extensively as part of our language learning app.

I was surprised to find that the speech part was not working on some of the latest Android devices, like Galaxy S8 - produced by Korea yet no Korean TTS voice installed by default. I hope the changes in this PR will help dealing with missing voice issues.

Copy link
Owner

@mihirsoni mihirsoni left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mihirsoni mihirsoni merged commit f3d2c75 into mihirsoni:master Jul 4, 2017
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.

None yet

2 participants