Skip to content

Commit

Permalink
temp: remove TTS Nag in Reviewer
Browse files Browse the repository at this point in the history
We want to remove legacy TTS, but the migration process is not
sufficiently documented.

While we get 2.17 out, remove the nag so we don't get review-bombed
more than necessary

This will be reverted once we have a better process

Issue 15475
  • Loading branch information
david-allison authored and lukstbit committed Feb 21, 2024
1 parent 9f49124 commit 95ab0de
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 8 deletions.
7 changes: 0 additions & 7 deletions AnkiDroid/src/main/java/com/ichi2/anki/ReadText.kt
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,6 @@ import android.speech.tts.UtteranceProgressListener
import android.view.WindowManager.BadTokenException
import androidx.annotation.VisibleForTesting
import androidx.appcompat.app.AlertDialog
import com.google.android.material.snackbar.BaseTransientBottomBar.LENGTH_INDEFINITE
import com.ichi2.anki.UIUtils.showThemedToast
import com.ichi2.anki.reviewer.CardSide
import com.ichi2.anki.snackbar.showSnackbar
Expand Down Expand Up @@ -289,12 +288,6 @@ object ReadText {
Timber.w("TTS not successfully initialized")
}
}
// Remember the user to upgrade to the new TTS system #15475
ankiActivityContext?.showSnackbar(R.string.readtext_reviewer_warn, LENGTH_INDEFINITE) {
setAction(R.string.scoped_storage_learn_more) {
ankiActivityContext.openUrl(R.string.link_tts)
}
}
}

fun errorToDeveloperString(errorCode: Int): String {
Expand Down
2 changes: 1 addition & 1 deletion AnkiDroid/src/main/res/values/10-preferences.xml
Original file line number Diff line number Diff line change
Expand Up @@ -293,7 +293,7 @@ this formatter is used if the bind only applies to both the question and the ans
<!-- Advanced -->
<string name="pref_cat_experimental" maxLength="41" tools:ignore="UnusedResources">Experimental</string>
<string name="readtext_deprecation_warn">AnkiDroid has introduced a better TTS mechanism which is compatible with other Anki clients and includes more voices and improvements to language playback!\n\nPlease upgrade as soon as possible, as this setting will be removed soon</string>
<string name="readtext_reviewer_warn">Please upgrade to the new text to speech format</string>
<string name="readtext_reviewer_warn" tools:ignore="UnusedResources">Please upgrade to the new text to speech format</string>

<!-- Developer options -->
<string name="pref_cat_dev_options" maxLength="41">Developer options</string>
Expand Down

0 comments on commit 95ab0de

Please sign in to comment.