Skip to content
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

Closed
wants to merge 5 commits into from
Closed

Spanish language support added #205

wants to merge 5 commits into from

Conversation

saketkumar
Copy link

@saketkumar saketkumar commented Nov 1, 2016

I have added the support for Spanish Language #169 . Please Check it.

@saketkumar saketkumar changed the title spanish language support added Spanish language support added Nov 1, 2016
@opticod
Copy link
Collaborator

opticod commented Nov 29, 2016

Hey @saketkumar95 , are you confident with your translation? Is this performed using any google translator ?

@saketkumar
Copy link
Author

Yes i'm confident with the translations. I have cross checked it 2-3 times using many google translator.

Copy link
Collaborator

@opticod opticod left a 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?

@saketkumar
Copy link
Author

@opticod I'm having some problem with this branch and want to remove. What's the status for this PR?

@opticod
Copy link
Collaborator

opticod commented Jan 2, 2017

@saketkumar95 I am waiting for the notification from @croozeus

@croozeus
Copy link
Member

croozeus commented Jan 2, 2017

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.

@croozeus croozeus closed this Jan 2, 2017
@saketkumar
Copy link
Author

Cool. No problem :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants