Page MenuHomePhabricator

Limit text to 300 chars and fix ssml if only passing "text" as a param
Closed, InvalidPublic2 Estimated Story Points

Description

After T314834: Make ipa optional inside of Phonos you can use phonos as a TTS engine for any text. We should limit the length of the text attribute just like we do with IPA.

I say 300 because that's what we currently limit the IPA string but I'm intrigued now how we came up with that since it seems very generous

Acceptance Criteria

  • Phonos should fail if you try to render a string over 300 characters long
  • SSML does not need to include a <phoneme> node