Skip to content

Conversation

@YashShah-Josh
Copy link

Update environment configuration to include Zaban API settings. Refactor audio service to utilize Zaban for speech-to-text and translation, replacing Sarvam. Adjust requirements and example usage accordingly.

…n. Update environment configuration to include Zaban API settings. Refactor audio service to utilize Zaban for speech-to-text and translation, replacing Sarvam. Adjust requirements and example usage accordingly.
Copy link
Collaborator

@Selectus2 Selectus2 left a comment

Choose a reason for hiding this comment

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

Please look at the given comments

… Zaban API paths and language mappings. Removed hardcoded values and improved maintainability by centralizing configuration in a new constants module.
@YashShah-Josh
Copy link
Author

Please look at the given comments

resolved comments, do check it once

Copy link
Collaborator

@Selectus2 Selectus2 left a comment

Choose a reason for hiding this comment

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

LGTM

@Selectus2 Selectus2 added the Ready to merge Approved and ready to merge and deply label Feb 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Ready to merge Approved and ready to merge and deply

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants