jovotech / jovo-framework

🔈 The React for Voice and Chat: Build Apps for Alexa, Messenger, Instagram, the Web, and more
https://www.jovo.tech
Apache License 2.0
1.67k stars 310 forks source link

Feature: Allow defining Alexa Skill permission settings in app.json #168

Closed serg06 closed 6 years ago

serg06 commented 6 years ago

I'm submitting a...

Expected Behavior

Alexa skills have a permission setting in skill.json which it looks like Jovo doesn't support yet:

"permissions": [
  {
    "name": "alexa::devices:all:address:full:read"
  }
]

(I got it by enabling the permission in UI then doing jovo get.)

We should be able to add it to our app.json file and, when we build, it should get added to platforms/alexaSkill/skill.json.

Current Behavior

Error log

If you have an error log, please paste it here.

Your Environment

aswetlow commented 6 years ago

Hey @serg06! You can put it to the alexaSkill.manifest object:

{
    "alexaSkill": {
        "nlu": "alexa",
        "manifest": {
            "permissions": [
                {
                    "name": "alexa::devices:all:address:full:read"
                }
            ]
        }
    },
    "endpoint": "https://webhook.jovo.cloud/<ID>"
}
serg06 commented 6 years ago

Oh wow, thanks!