-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
feat(jigasi,transcriber): transcriber component and jigasi configuration updates #1881
Conversation
25585fc
to
9d86242
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe I've made the requested changes and hope to have another review
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not a docker guru, but spotted a few jigasi-related config problems.
8eb5b37
to
659cc15
Compare
a3c7c7f
to
3c0a039
Compare
ok I believe I have trimmed down all the extraneous changes here, and would love a review from @saghul once he's returned and has a brain for such. I have tested the transcriber part in compose and in nomad and both eventually worked. I have not tested the sip part since the changes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Excellent work! 👏
No description provided.