-
Notifications
You must be signed in to change notification settings - Fork 145
Add example utterances to AMAZON.HelpIntent. #216
Conversation
|
There are several things in this PR that require German translations:
Ping @mcl22 and @ausweider :) |
|
@mcl22 and @ausweider, I forgot to mention.. the values for these in That is, they should ideally all be from the same album and by the same artist, otherwise you could get some weird suggestions like: ;) |
|
Ok, here we go. I hope I didn't miss anything :) template strings: utterances: sample slotvals: |
|
Thanks again @mcl22 :) |
|
Just a thought, but given the double meaning of the word "alternative" is it going to be a little confusing when used in context? ListenToGenre listen to alternative genre ListenToArtist listen to alternative artist Sia |
|
It doesn't sound weird to me, but honestly it doesn't matter to me either what it is so long as it fits, so I'll change it :) |
|
Have you tried this PR with AWS? If no one gets around to it, I can do it, but it'd save me some trouble. |
|
Yes, I can add quotes around them. I actually hadn't looked at it using the mobile app. As for the list of things, I prefer a random set because it shows variations. Certain intents can be excluded, however. I've already excluded the PlayerZoom intents and had considered excluding the navigation commands because things like "right" without any context are kind of confusing, particularly in the spoken response. edit: there should be newlines there too.. Amazon says they can be included, and they show up right in a web browser.. not sure why they're not there on mobile. |
|
Had to insert a non-breaking space in between two newlines to get it to show an empty line on the mobile app.. kinda weird, but it works. Lemme know if that looks better. |
|
As an aside, it'd be nice if Amazon allowed hypertext links, though I can understand why they don't (security risk). |
|
Thoughts on including the navigation/system commands? Particularly these: |
|
That's fine with me |
|
Meaning you'd prefer to include or exclude them? |
|
Sorry, I think it's fine to include those. |
|
Ok, all that needs to be done I think then is to test on Heroku. |
|
I'll give it a test real quick |
|
Works great on Heroku |
|
Awesome, thanks for checking. |
Thanks to @mcl22 for the German translations.
This expands the Help response to include random example utterances the user can try.
The spoken response includes one random utterance and another for the reprompt. The same reprompt is used for
ask.launch(skill opened with no Intent requested, as in,open kodi).The Card presented in the Alexa app provides 5 randomly-chosen utterances.
The example utterances are chosen from the
speech_assets/SampleUtterances.LANG.txtfile. A new filesample_slotvals.LANG.txtwas created to fill slot references with something to make it sound more natural than something likeinsert title here.Needs testing for Heroku deployments