search

declension / squeeze-alexa

Squeezebox integration for Amazon Alexa
GNU General Public License v3.0
59 stars 20 forks source link

added new version of intent to that works with intent v1 model #61

Closed NadirJ closed 6 years ago

NadirJ commented 6 years ago

fixed #23

Based on some quick tests it looks like this new intent v1 json is working. (using US English)

Note:

I'm not sure how this will work with the german

dougadams commented 6 years ago

Alexa and Lambda noob.

Working on implementing squeeze-alexa.

Thanks for this file. I wasn't sure how to structure the json and txt files in the HowTo with the current skill builder. A great help.

It wouldn't save as is though. Error message that the type "PLAYLIST" was missing synonyms. I deleted the blanks

in the Playlist type: {"name": {"value": ""}},

and was able to save.

Now on to the next step.

dougadams commented 6 years ago

I also couldn't complete the build with the custom samples in the AMAZON.ShuffleOffintent". Build failed. When I deleted the samples for that intent the build was successful.

NadirJ commented 6 years ago

Sorry about the empty playlists values...I used a quick regex on the original txt file to get build the json and didn't check my work (I'm using my own playlist values). I also fixed up the ShuffleOffIntent

declension commented 6 years ago

Thanks @NadirJ Been very busy but I plan to get some time to look at all this (plus how this will work internationalised)

declension commented 6 years ago

@NadirJ I've updated master a whole bunch to (hopefully) sort out i18n in a better way that will support v0 and v1 interaction schemas depending on user choice / translation progress.

Now looks like this:

metadata
└── intents
    └── v0
        └── locale
            ├── de_DE
            └── en_GB
                └── slots

Do you think you could update this PR to move your new file into a v1/locale/en_US directory (or en_GB, hard to say which it really is)?

Ideally mention this new setup in the HOWTO section too, but I can do that after merge if you prefer.

NadirJ commented 6 years ago

I'll take care of updating the PR. I've only tested en_US so it makes sense to put it in v1/locale/en_US

NadirJ commented 6 years ago

sorry for the delay I moved the file and updated the HOWTO

declension commented 6 years ago

Thanks @NadirJ :+1: :+1: :+1: