gabriele-v / hubitat

Apache License 2.0
10 stars 21 forks source link

Brazilian Portuguese support #3

Open MAFFPT opened 4 years ago

MAFFPT commented 4 years ago

Hi Gabriele!

Not a long time ago I suggested a modification to the Alexa TTS to include the Alexa Brazilian Portuguese support. It was accepted and I did the change and it works fine.

However a few days ago, when I was checking your code to understand how it works, I could see that there is no support for that language in your code.

So, I'm suggesting including the following line of code right after the line 111 to include Brazilian Portuguese support:

<input type="radio" name="amazonSite" value="alexa.amazon.br|pt-BR">Brazil<br>

Also, I've noted that in the same line of code that there is something that I did not understand. Let's look at the line 111:

<input type="radio" name="amazonSite" value="alexa.amazon.ca|en-US"> Italy<br>

The site for Alexa is Canada (suffix .ca) and not Italy (suffix .it), and also the language is coded English US (en-US), and I think that it should be Italian (it-IT). Forgive me if I'm wrong!

Best,

Marco