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

nodejs-es: Sync site.json with english version #1836

Merged
merged 3 commits into from
Oct 22, 2018

Conversation

Tsuna-mi
Copy link
Contributor

This PR update locale/es/site.json file as its english version.
Thanks for reviewing it! :)

Related with #1778
cc @nodejs/nodejs-es

This PR update locale/es/site.json file as its english version.
Thanks for reviewing it! :)
Related with #1778 
cc @nodejs/nodejs-es
@Tsuna-mi Tsuna-mi added the i18n Issues/PRs related to the Website Internationalisation label Oct 13, 2018
}
},
"getinvolved": {
"link": "get-involved",
"text": "Participa",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think "Participa" makes more sense in this context rather than "Implícate"

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tried it with 'implícate' cause is the word used at catalan translation. I can change in both files by 'Participa' to be consistent.
Thanks @academo

Copy link
Contributor

@franher franher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@fhemberger
Copy link
Contributor

@Tsuna-mi Could you please check for merge conflicts? Would like to get this merged.

@Tsuna-mi
Copy link
Contributor Author

@Tsuna-mi Could you please check for merge conflicts? Would like to get this merged.

@fhemberger Sorry I don't see conflicts.. I've fixed last week
captura de pantalla 2018-10-22 a las 18 14 08

@fhemberger
Copy link
Contributor

@Tsuna-mi Ok, may be related to GitHub's hiccup today. Please go ahead and merge this PR, I can't do it myself:

bildschirmfoto 2018-10-22 um 18 37 09

@Tsuna-mi Tsuna-mi merged commit a5b169f into master Oct 22, 2018
@Tsuna-mi Tsuna-mi deleted the Tsuna-mi-sync-site-json-es branch October 22, 2018 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
i18n Issues/PRs related to the Website Internationalisation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants