Pizzaface / Alexa-Chromecast-Skill-2.0

Allows Amazon Alexa to control Google Chromecast - Designed for Raspberry Pi
360 stars 63 forks source link

Spanish version of Alexa Chromecast Skill 2.0 #41

Closed cristian44137 closed 3 years ago

cristian44137 commented 3 years ago

I have made an Spanish compatible version of this repository, I tested it and it works as a charm. I hope someone can tell me if they see any error, as I started coding this without any experience. Firstly I created it to my own purposes, but it would be great if someone more can enjoy it without having to make a rough translation.

LukeChannings commented 3 years ago

Hey @CrStN224,

Thank you so much for your work on this, making software more multilingual is important.

Unfortunately it's also complicated, which makes merging this request difficult. If I were to merge this request, the language would be in Spanish for everyone, which is probably not what you intended.

My suggestions are:

Another option would be to have a Spanish branch, and update the readme to link to the Spanish branch instead of README.es.markdown.

cristian44137 commented 3 years ago

That's right, I thought about creating a Spanish branch, this way it would be easier to compare or merge new changes on both branches, and wouldn't have to remove the changes made in the shell. Anyway, tell me what do you prefer. I will update the pull request and files to your conditions.

LukeChannings commented 3 years ago

The quickest and easiest solution would probably be to have a spanish branch, with a link to that in the README. You'll have to be responsible for keeping it up-to-date with master, though.

cristian44137 commented 3 years ago

No problem, I will try my best. I'm receiving notifications to my phone whenever you make a change.

LukeChannings commented 3 years ago

Ok thanks, you'll need to:

cristian44137 commented 3 years ago

Okay, could you please create the new branch? I have not managed to create it for myself, as I have searched how to do it but found nothing. I will update the MR and choose the new branch you created, and then push the MR updating the readme file with an url that redirects to the spanish branch for Spanish speakers.

LukeChannings commented 3 years ago

I've just pushed a spanish branch, hopefully you can update your MR now.

LukeChannings commented 3 years ago

I've just quickly added a link, no need for another PR.

Thanks for your contribution!

cristian44137 commented 3 years ago

Great! You're welcome, thank you too!