-
Notifications
You must be signed in to change notification settings - Fork 130
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
Spanish language support added #205
Conversation
Hey @saketkumar95 , are you confident with your translation? Is this performed using any google translator ? |
Yes i'm confident with the translations. I have cross checked it 2-3 times using many google translator. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@croozeus , can we merge this?
@opticod I'm having some problem with this branch and want to remove. What's the status for this PR? |
@saketkumar95 I am waiting for the notification from @croozeus |
Google translations is not the correct way of doing this. They are good for a start but to merge into master - we'll need couple of native spanish speakers to review this. Moreover, at a project level - I think the best way to do translations is to use a crowdsourced service - I'll look more into it this month. Closing this for now. |
Cool. No problem :) |
I have added the support for Spanish Language #169 . Please Check it.