Skip to content
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

Example which creates output for use with new Skill Builder #322

Merged
merged 5 commits into from
Mar 17, 2018
Merged

Example which creates output for use with new Skill Builder #322

merged 5 commits into from
Mar 17, 2018

Conversation

funkydan2
Copy link
Contributor

Putting <%=app.schemas.skillBuilder()%> in an ejs file seems to create the output which can be used with the new Skill Builder.

(Sorry, I'm not super familiar with using GitHub, hopefully this PR is in the right format to be useful.)

Added an example of how to output the JSON required for the new SkillBuilder.

I'm new to this, so I'm not sure if this is the right & best way, but it works for me.
CHANGELOG.md Outdated
@@ -4,6 +4,7 @@

* [#315](https://github.com/alexa-js/alexa-app/pull/315): Fix card tests and required card type - [@kobim](https://github.com/kobim).
* [#314](https://github.com/alexa-js/alexa-app/pull/314): Fix handling of `undefined` slot values - [@User1m](https://github.com/user1m).
* Included Skill Builder example in example.ejs - [@funkydan2](https://github.com/funkydan2)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can you please make it look like the other changes here? Include the PR number and a period in the end.

@funkydan2
Copy link
Contributor Author

Sorry, @dblock is this closer to the formatting required?

@dblock dblock merged commit 1df3ac8 into alexa-js:master Mar 17, 2018
@dblock
Copy link
Collaborator

dblock commented Mar 17, 2018

Merged, thanks.

@matt-kruse
Copy link
Collaborator

Just catching up to this, but I've been using this in my "deploy" script and it works well:

node -e "console.log(require('./src/index.js').app.schemas.askcli())" > models\en-US.json

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants