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

Feature/bugfix TTS provider settings #3778

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

arben-adm
Copy link

Fix Bug: Unable to Add or Update Speech-to-Text Provider in Flow Settings

Description:
Ever since the groq update, I’ve encountered a bug (reproducible on both Windows and Linux) where attempting to add or change the speech-to-text provider under Settings → Configuration → Speech to Text within a flow does nothing. Clicking the Save button multiple times has no effect—the settings are simply not applied.

I’m submitting this fix to resolve the issue. Below are the key changes I’ve made:

State Management: Ensures state updates are handled properly.
Provider Selection: Fixes the toggling mechanism so the chosen provider is updated correctly.
Configuration Persistence: Makes sure saved settings actually persist after clicking Save.
Validation: Adds proper checks before persisting new settings.
User Feedback: Provides clear notifications through snackbars, improving UX.

The core problem appears to have been in how the state was managed, and the configuration prepared prior to saving. With these modifications, adding or updating a speech-to-text provider should now work as expected.

First PR, i hope this will be helpful for many! :-)

grafik

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.

1 participant