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

Resolves #17543 TTS availability among devices #17561

Closed
wants to merge 5 commits into from

Conversation

Ssdosaofc
Copy link
Contributor

Resolves #17543 Added Text-to-Speech engine availability check, accounting for devices like Amazon Fire Stick.
Well, here I might need some help implementing all string values for "tts_unavailable_title" and "tts_unavailable_message." :)

Copy link
Contributor

github-actions bot commented Dec 5, 2024

Important

Maintainers: This PR contains Strings changes

  1. Sync Translations before merging this PR and wait for the action to complete
  2. Review and merge the auto-generated PR in order to sync all user-submitted translations
  3. Sync Translations again and merge the PR so the huge automated string changes caused by merging this PR are by themselves and easy to review

@lukstbit
Copy link
Member

lukstbit commented Dec 5, 2024

Hey, this has commits from your previous PR which makes it harder to see what are the real changes. You should git rebase on top of main to only see here the commit with the actual changes.

@mikehardy
Copy link
Member

probably best to work off branches as well - I see you are using your main branch instead of feature development forks in your repo, that will contribute to this sort of messy commit history. Better to keep your main in sync with upstream main, and always branch fresh for a new feature

With regards to language, don't worry about this: only the main values/ folder needs changes, the rest are automatically populated from crowdin, our translation platform

@Ssdosaofc
Copy link
Contributor Author

Thank you for pointing it out. Now it looks fine.

Comment on lines +81 to +92
// check whether tts is available or not first
val ttsAvailable = checkTtsEngine()
if (!ttsAvailable) {
AlertDialog.Builder(requireContext()).show {
setTitle(R.string.tts_unavailable_title)
setMessage(R.string.tts_unavailable_message)
setIcon(R.drawable.ic_warning)
setPositiveButton(R.string.dialog_ok) { _, _ -> }
}
ttsPref.isChecked = false
return@setOnPreferenceChangeListener false
}
Copy link
Member

Choose a reason for hiding this comment

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

This feature is intended to be deleted.

We need to handle the unified mechanism of TTS:

https://docs.ankiweb.net/templates/fields.html#text-to-speech-for-individual-fields

abstract class TtsPlayer : Closeable {

@david-allison david-allison added the Needs Author Reply Waiting for a reply from the original author label Dec 9, 2024
Copy link
Contributor

Hello 👋, this PR has had no activity for more than 2 weeks and needs a reply from the author. If you think this is a mistake please comment and ping a maintainer to get this merged ASAP! Thanks for contributing! You have 7 days until this gets closed automatically

@github-actions github-actions bot added the Stale label Dec 23, 2024
@github-actions github-actions bot closed this Dec 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Author Reply Waiting for a reply from the original author Stale Strings
Projects
None yet
Development

Successfully merging this pull request may close these issues.

TTS: Better handle the case where no Engines are available
4 participants