-
-
Notifications
You must be signed in to change notification settings - Fork 11
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Google Actions: AMAZON.YesIntent is invalid #20
Comments
Hey! Could you paste the relevant part of your model JSON? Is the |
Ah yeah, they're definitely in the common area. Just realized there was a different layer of our stack that was responsible for normalizing these. Sadly only 1/3 of the issues I filed friday ended up being valid lol |
I guess the use case we have is that we just use names like
`AMAZON.YesIntent` in the common section of our model file. DialogFlow has
always been happy with those names, even though it looks weird. There's an
argument to be made that we should have just called it `YesIntent` with a
platform override and that it's good that the platform is calling us out
for it.
…On Tue, Oct 13, 2020 at 11:45 PM Alex ***@***.***> wrote:
Good question. Does it make sense to have periods in the intent name? Even
Alexa doesn't support periods in custom intents.
Not sure if it would break existing voice apps
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
<#20 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAEERGAH7PI3J3USMSFPQZDSKVCILANCNFSM4SCBOUJA>
.
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I believe the jovo-cli on dialogflow used to take care of normalizing these intents into just
YesIntent
etc. I'm now rolling my own CLI by utilizing jovo-model.I'm not sure if that falls within the scope of this library or if that's something I should be handling in my own code. Would love some clarity!
The text was updated successfully, but these errors were encountered: