MycroftAI / skill-speak

Mycroft AI official Speak Skill - make Mycroft speak back text
https://mycroft.ai/skills
Apache License 2.0
5 stars 25 forks source link

Case issue and no corresponding word in some translation. #18

Closed Nemirtingas closed 3 years ago

Nemirtingas commented 5 years ago

When you use cli, if you use an uppercase beginning letter (like Say), then it doesn't get removed in the result. Say Hi => Say Hi say Hi => Hi

Also there's a bug in the french translation: in regex/fr-fr/Words.rx there are only 2 different triggers: 'dis' and 'répète' dis (?P.) <------ Replace this as 'parle' dis (?P.) répète (?P.*)

But in vocab/fr-fr/Speak.voc there are 3 of them: 'parle', 'dis', 'répète' parle dis répète

forslund commented 5 years ago

Hi @Nemirtingas I'll look over the regexes, as you say they should be case instensitive.

For the missing parle in the french translation feel free to submit this change at https://translate.mycroft.ai