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

fa: translate locale Persian "fa" (#1866) #1874

Merged
merged 8 commits into from
Nov 1, 2018
Merged

fa: translate locale Persian "fa" (#1866) #1874

merged 8 commits into from
Nov 1, 2018

Conversation

m-esm
Copy link
Contributor

@m-esm m-esm commented Nov 1, 2018

site.json and some of website main pages translated to Persian "fa" language.

@@ -1,19 +1,19 @@
{
Copy link
Contributor

Choose a reason for hiding this comment

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

Could you pease check if the translations here for site.json against those in #1871?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes there are against #1871

reviews considered in the last commit and more parts translated.

Copy link
Contributor

Choose a reason for hiding this comment

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

So we can merge this PR instead of #1871?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes. 😃

@fhemberger
Copy link
Contributor

/cc @nodejs/nodejs-fa Please review the translation. Depending on the complexity of changes, a review of one or two people is recommended. If everything is okay, click "Approve". Then we know it's ready to be merged into master.

locale/fa/about/index.md Outdated Show resolved Hide resolved
locale/fa/about/index.md Outdated Show resolved Hide resolved
locale/fa/about/index.md Outdated Show resolved Hide resolved
locale/fa/security.md Outdated Show resolved Hide resolved
locale/fa/security.md Outdated Show resolved Hide resolved
locale/fa/security.md Outdated Show resolved Hide resolved
locale/fa/security.md Outdated Show resolved Hide resolved
locale/fa/security.md Outdated Show resolved Hide resolved
locale/fa/security.md Outdated Show resolved Hide resolved
@ebraminio
Copy link

LGTM with some nits

locale/fa/about/index.md Outdated Show resolved Hide resolved
locale/fa/about/index.md Outdated Show resolved Hide resolved
locale/fa/about/index.md Outdated Show resolved Hide resolved
locale/fa/about/index.md Outdated Show resolved Hide resolved
locale/fa/about/index.md Outdated Show resolved Hide resolved
locale/fa/index.md Outdated Show resolved Hide resolved
locale/fa/security.md Outdated Show resolved Hide resolved
locale/fa/security.md Outdated Show resolved Hide resolved
locale/fa/security.md Outdated Show resolved Hide resolved
locale/fa/security.md Outdated Show resolved Hide resolved
locale/fa/security.md Outdated Show resolved Hide resolved
locale/fa/security.md Outdated Show resolved Hide resolved
locale/fa/security.md Outdated Show resolved Hide resolved
Copy link

@ebraminio ebraminio left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks :)

locale/fa/security.md Outdated Show resolved Hide resolved
locale/fa/security.md Outdated Show resolved Hide resolved
locale/fa/security.md Outdated Show resolved Hide resolved
@fhemberger
Copy link
Contributor

@m-esm @ebraminio Please let me know when you're done, everything is reviewed and you're good to merge.

Copy link

@ebraminio ebraminio 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 fhemberger merged commit dab02b1 into nodejs:master Nov 1, 2018
@fhemberger
Copy link
Contributor

Thank you!

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.

4 participants