Improve indication of disabled TTS engines

bertfrees commented 3 months ago

Currently there is a message underneath the "Key" and "Region" fields saying "Property not set: org.daisy.pipeline.tts.azure.key". This is the message that comes from the engine. From inside the UI this message makes somewhat less sense, and could be improved. I'm not sure exactly how, but we could use the following:

The engine returns the "disabled" status for this engine: this is a special state that differs from "error" (intended/less severe errors). We could use this information to do something different.
If a message contains a property name, perhaps we could use this information to point to the input field that is not set. Alternatively we could somehow mark properties in the (/admin/properties API) that are required for a TTS engine to work.

marisademeglio commented 3 months ago

We know that key and, if applicable, region are the required fields for the TTS engines form so we can build in some feedback to the user that they have to be filled in for the connection to work.

marisademeglio commented 3 months ago

I did some work on this, see the test build here https://github.com/daisy/pipeline-ui/actions/runs/9726989779

bertfrees commented 3 months ago

Looks good!

daisy / pipeline-ui

Improve indication of disabled TTS engines #235