Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

Speech integration #6

Merged
merged 16 commits into from
Jul 2, 2018
Merged

Speech integration #6

merged 16 commits into from
Jul 2, 2018

Conversation

gurion
Copy link
Contributor

@gurion gurion commented Jun 28, 2018

No description provided.

@gurion gurion requested review from zisimo and als2076 June 28, 2018 14:59
app.js Outdated
@@ -34,6 +34,16 @@ bot.recognizer(recognizer);
// Configure PMKB Client. ENV variables are stored in Azure.
const pmkbClient = new PMKBClient(process.env.PMKB_HOST, process.env.PMKB_USER, process.env.PMKB_PASS);

// Set up speech services
const speechOptions = {
speechRecognizer: new CognitiveServices.SpeechRecognizer({ subscriptionKey: 'MICROSOFT_SPEECH_API_KEY' }),
Copy link
Contributor

Choose a reason for hiding this comment

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

MICROSOFT_SPEECH_API_KEY
Doesn't that come from environment variable?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yeah, I thought I changed that sorry

app.js Outdated
speechRecognizer: new CognitiveServices.SpeechRecognizer({ subscriptionKey: 'MICROSOFT_SPEECH_API_KEY' }),
speechSynthesizer: new CognitiveServices.SpeechSynthesizer({
gender: CognitiveServices.SynthesisGender.Female,
subscriptionKey: 'MICROSOFT_SPEECH_API_KEY',
Copy link
Contributor

Choose a reason for hiding this comment

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

same with the comment above

Copy link
Contributor Author

Choose a reason for hiding this comment

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

^

@zisimo zisimo merged commit 8db9f65 into develop Jul 2, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants