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

sample utterances (w/optional slots, like +GENRE+ and +TIVOBOXRM+) #24

Closed
greenpad2 opened this issue Feb 19, 2017 · 5 comments
Closed
Assignees
Labels

Comments

@greenpad2
Copy link

When I went to copy/paste sample utterances, I found that a good number of them had +SOME_STRING+ in them, such as +GENRE+. Amazon didn't like this, so I just got rid of the plus signs. Is this a sign I left something out when I got this all running? I have a feeling I lost some functionality by changing all of those.

Thanks!

@jradwan jradwan self-assigned this Feb 19, 2017
@jradwan
Copy link
Owner

jradwan commented Feb 19, 2017

Yes, this is due the addition of optional slot utterances by @dominicankev, which requires this pull request for alexa-utterances. That PR hasn't been merged into the main alexa-utterances code yet, so for now you'll need to make that change manually. I mention this in the release notes for v0.4.0 but it's easily missed.

If you don't want to modify alexa-utterances, just replace the + with the corresponding curly brace. For example:

ListChannels for my channels by +GENRE+ becomes ListChannels for my channels by {GENRE}

and

PutOn put on {CHANNELNAME} in +TIVOBOXRM+ becomes PutOn put on {CHANNELNAME} in {TIVOBOXRM}

If you don't do this, all you lose is the optional utterances used for one-time channel changes on a TiVo in another room without having to switch to it first (i.e., "Alexa, tell TiVo to put on ESPN in Family Room") and listing channels by genre (i.e., Alexa, tell TiVo to list my channels by comedy").

Let me know if that works for you.

@jradwan jradwan closed this as completed Feb 19, 2017
@jradwan jradwan reopened this Feb 19, 2017
@jradwan jradwan changed the title sample utterances sample utterances (w/optional slots, like +GENRE+ and +TIVOBOXRM+) Feb 19, 2017
@greenpad2
Copy link
Author

Thanks! I'll go change things to curly braces. I'm not sure I understand the purpose of the plus signs but I'll keep reading.

@jradwan
Copy link
Owner

jradwan commented Feb 19, 2017

Using the plus signs was just @dominicankev's method of marking a slot as optional for alexa-utterances. His PR is the code that changes the + to the { or } for when they appear in the Alexa Tester to copy into the skill's utterances.

@jradwan
Copy link
Owner

jradwan commented Nov 29, 2017

Here are the required manual code changes.

@jradwan
Copy link
Owner

jradwan commented Feb 13, 2018

Since the PR still isn't merged, for now I'm including a patched version of the index.js file for alexa-utterances in the examples directory. I updated Step 6 of the installation instructions to include applying this patch.

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

No branches or pull requests

2 participants