Merged
Conversation
Contributor
Author
|
waiting on jambonz/time-series#18 |
Contributor
Author
|
There's an additional fix on Line 1253 of Call-session, we were always using the STT_NOT_PROVISIONED alert type even if the call was using a Say verb, as the initial validation is on all speech providers together, so we now check what type its testing and raise the appropriate alert |
Contributor
|
can you rebase? |
Contributor
Author
|
@davehorton done |
davehorton
approved these changes
Dec 11, 2025
zscgeek
added a commit
to consigai/jambonz-feature-server
that referenced
this pull request
Dec 17, 2025
….9.5-8a * commit 'c5cd488fdfb40ae135d07fefcd169b7d3d01381d': (107 commits) fixed gather should ignore transcription if task is killed/resolved. (jambonz#1465) add label to STT/TTS alerts (jambonz#1468) add support for sending DTMF to ultravox (jambonz#1471) fix jambonz#1466: (jambonz#1467) Add configurable say chunk size (jambonz#1461) Enhance TTS sentence boundary detection for Arabic and Japanese (jambonz#1464) fixed dial verb cannot bridge 2 leg endpoints due to transcoding (jambonz#1457) fixed undefine issue when setting tts streaming channel vars (jambonz#1456) when dialing the B leg we check to see if we are using opus on the A leg, and if so we outdial B with opus first; however we were incorrectly checking the SDP on the A leg invite not the 200 OK we send back (jambonz#1455) use timeout on HTTP requests (jambonz#1453) Add ability to enable/disable Azure audio logging via azureOptions (jambonz#1432) SoundHound support audio endpoint from speech credential (jambonz#1446) Fixed transcribe 2 legs cannot fallback (jambonz#1451) fix race condition where gather resolves with speech transcript but t… (jambonz#1449) Compare sdp to determine if transcoding is being used. (jambonz#1444) allow say verb failed as NonFatalTaskError for File Not Found (jambonz#1443) fixed playbackIds is not in correct order compare with say.text array (jambonz#1439) fix say verb does not close streaming when finish say (jambonz#1412) Add ability to override certain tts streaming options via the config … (jambonz#1429) fix bug escaping backspace character ... # Conflicts: # lib/utils/background-task-manager.js # package-lock.json # package.json
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add the label to the alert message for TTS/STT not configured,
Setting the label on a speech provider Then not using that lable in the verb is a common user error, by adding the label (or None) to the alert message this should prompt users to check the label config.
Example alert messages for both the label set in the verb and not configured (TestLabel) and where the provider has a label but not set in the verb (none)
Created as draft as this also requires an update to @jambonz/time-series first