Using the --async flag for alexa deploys calls getImportStatus right after deploying the skill. My testing showed that this leads to the response not being complete, see example:
As you can see the result here only contained status, while the skill property is not there. As Jovo is trying to access status.skill.skillId, this leads to the error in the screenshot.
I found out that this seems to be dependent of the time passed since the import. Waiting a moment before requesting the import status fixes this issue, see this image that shows the status result:
This is only relevant for --async deploys, due to non async deploys ignore IN_PROGRESS responses already.
I was not sure whether to put this line of code before the getImportStatus call as it is now, or inside getImportStatus. I think it's a matter of preference, feel free to change or request changes if necessary.
Types of Changes
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
Checklist
[x] My code follows the code style of this project
[ ] My change requires a change to the documentation
Proposed Changes
Using the
--async
flag for alexa deploys callsgetImportStatus
right after deploying the skill. My testing showed that this leads to the response not being complete, see example:As you can see the result here only contained
status
, while theskill
property is not there. As Jovo is trying to accessstatus.skill.skillId
, this leads to the error in the screenshot.I found out that this seems to be dependent of the time passed since the import. Waiting a moment before requesting the import status fixes this issue, see this image that shows the
status
result:This is only relevant for
--async
deploys, due to non async deploys ignoreIN_PROGRESS
responses already.I was not sure whether to put this line of code before the
getImportStatus
call as it is now, or insidegetImportStatus
. I think it's a matter of preference, feel free to change or request changes if necessary.Types of Changes
Checklist