jovotech / jovo-cli

🛠 Command Line Interface for the Jovo Framework: Makes voice experience deployment a breeze, including features like local development and staging.
https://www.jovo.tech/marketplace/jovo-cli
Apache License 2.0
57 stars 31 forks source link

Platform-specific intents incorrect on build #95

Closed sdaoud closed 4 years ago

sdaoud commented 5 years ago

I'm submitting a...

Expected Behavior

Running latest Jovo V2, in my language model file, I have this:

"intents": [
  {
    "name": "CatchAllIntent",
    "phrases": [
      "{query}"
    ],
    "inputs": [
      {
        "name": "query",
        "type": {
          "dialogflow": "@sys.any",
          "alexa": "CatchAll"
        }
      }
    ]
  }
]

Running jovo build should create a CatchAllIntent intent for both Alexa and DialogFlow. Dialogflow should use system entity @sys.any whereas Alexa should use the custom CatchAll entity (not shown here).

Current Behavior

If I run jovo build the dialogflow platforms folder shows the CatchAllIntent using the CatchAll entity, rather than @sys.any.

If I run jovo build --platform googleAction the CatchAllIntent for dialogflow uses @sys.any as expected.

Your Environment

aswetlow commented 5 years ago

Thanks! I can reproduce it.

Fixed it and will publish it soon.

Workaround for now: Run jovo build for each platform separately.

> jovo build --platform alexaSkill
> jovo build --platform googleAction
rubenaeg commented 4 years ago

Closing this due to inactivity. If the bug still persists, please reopen the issue.