-
Notifications
You must be signed in to change notification settings - Fork 2
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
Added sample #5
Added sample #5
Conversation
const voxaOpearlo = require('voxa-opearlo'); | ||
|
||
const opearloConfig = { | ||
userId: 'tkT8PClpPxMOvPbadGyzOoJKwoX2', |
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.
this shouldn't be here! let's add this to a configuration key and leave it empty with instructions in the readme about it
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.
Remember this is a public project, we can't expose ANY type of credentials
const voxaOpearlo = require('voxa-opearlo'); | ||
|
||
const opearloConfig = { | ||
userId: 'tkT8PClpPxMOvPbadGyzOoJKwoX2', |
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.
Remember this is a public project, we can't expose ANY type of credentials
|
||
return skill.execute(event) | ||
.then((reply) => { | ||
expect(reply.toJSON().response.outputSpeech.ssml).to.equal('<speak>Welcome!</speak>'); |
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.
let's either remove the tests from the sample or update them to actually pass
No description provided.