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

Add Chinese translations #456

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Add Chinese translations #456

wants to merge 6 commits into from

Conversation

rgommers
Copy link
Member

This integrates the Crowdin translations into master, fixes a number of rendering issues, and makes Chinese translations show up in PR previews (but not yet on numpy.org, we haven't launched yet).

In the PR preview it is now easy to see where there are still gaps in translations.

Note, to build the site locally with translations, use:

export NUMPYORG_WITH_TRANSLATIONS=1
python gen_config.py
hugo
hugo server

@rgommers rgommers requested a review from Qiyu8 July 23, 2021 19:36
@rgommers
Copy link
Member Author

Showing up now in preview:

image

sidebar: false
---

### 2021 NumPy survey
Copy link
Member

Choose a reason for hiding this comment

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

The news page has been override by @InessaPawson 's "Make an announcement about the 2021 NumPy survey", Not only the Chinese translation is affected, but the translation of other languages is also affected. should we fix this content? It's looks like the translation process is conflicted with document maintenance.

Copy link
Member

@melissawm melissawm Jul 26, 2021

Choose a reason for hiding this comment

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

Ah, I think this is what I mentioned at #55 (comment)

@charris
Copy link
Member

charris commented Apr 12, 2022

What is the status of this?

@rgommers
Copy link
Member Author

What is the status of this?

It's kind of stuck on lack of bandwidth from me to finish it. Merging this PR seems not too far off; just one review comment to deal with. Actually launching the translations on the public site is a bit more involved - and also I'm expecting that we'll get a larger amount of contributions at that point, and I at least have no bandwidth to respond to them at the moment. @InessaPawson and I talked recently about how to get translations moving; it'll take a few months probably.

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

Successfully merging this pull request may close these issues.

4 participants