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

WP-Translations updates #707

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from
Open

Conversation

fxbenard
Copy link

@fxbenard fxbenard commented Nov 3, 2017

Our Team and all our contributors are happy to support your project

# Conflicts:
#	languages/tgmpa-ja.mo
#	languages/tgmpa-ja.po
#	languages/tgmpa-pt_PT.mo
#	languages/tgmpa-pt_PT.po
#	languages/tgmpa-tr_TR.mo
#	languages/tgmpa-tr_TR.po
@jrfnl
Copy link
Contributor

jrfnl commented Nov 8, 2017

@fxbenard This is awesome ! Thank you so much. It may be a little while before we get round to reviewing & merging this, but please know that we really appreciate your support and these translations.

@jrfnl
Copy link
Contributor

jrfnl commented Mar 12, 2018

Hi @fxbenard Again, sorry for the delay before I had a chance to look at this properly and thanks again for your support.

Now, there are a couple of things I'd like to discuss with you:

  1. Before anything else, the PR needs to be rebased. Sorry about that.

  2. You appear to have updated the .pot file and the file headers (and line nrs) of all existing translation files.
    Some of the information now included in the file headers is incorrect.
    Also, for a number of translations, it looks like the actual translations haven't been updated at all, just the .po file synced with the updated .pot file. This is undesirable as it makes reviewing the PR a lot more difficult.

    Generally speaking, so far, we would do an update to the .pot file just before a release and update the .po files to match. We would then invite the translators who have been previously involved to review the updated files for their language and add/update translations where necessary before the new version of TGMPA is released.

    If you think that process can be improved, please let me know and I'd be happy to discuss it.

  3. The ca (Catalan) and nb_NO (Norwegian Bokmal) translations included in the PR are very incomplete and not suitable yet for inclusion in the project. As a general rule of thumb, at least 90% of the text phrases should be translated before a translation will be included.
    The zh_CN (Mandarin Chinese) translation and the da_DK (Danish) translation are both much closer to that mark, but also would not make it yet.

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.

2 participants