keatontaylor / alexa-actions

A README and associated code to get actionable notifications setup for Alexa devices.
GNU General Public License v3.0
391 stars 186 forks source link

[Bug]: YesIntent not working in portuguese #200

Closed gosoares closed 1 year ago

gosoares commented 1 year ago

Did you check our FAQ and Issue/PR for similar issues?

Contact Details

soaresggm@gmail.com

What happened?

The word "sim" (yes in portuguese) is recognized as a String Intent instead of a AMAZON.YesIntent. sáb 02 set 2023 19:45:49 -03

After a few tests, i was able to fix this by change the slot type of the string intent to "AMAZON.FirstName". With this the "sim" is correctly assigned to the YesIntent while other texts still are recognized as the String Intent. I could do a PR with this change, if this is ok.

Version

latest

What Alexa device are you using?

Echo, Echo Show, Fire TV

What other Alexa device are you using?

No response

Relevant log output

No response

DEADSEC-SECURITY commented 1 year ago

Yes, please go ahead and create a PR with this fix