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 Japanese translation #368

Merged
merged 5 commits into from
Jun 16, 2022
Merged

Add Japanese translation #368

merged 5 commits into from
Jun 16, 2022

Conversation

Ippey
Copy link
Contributor

@Ippey Ippey commented Jun 14, 2022

Type of Change

  • Something else: Add Japanese translation.

What issue does this relate to?

None

What should this PR do?

  • Add Japanese translation.
  • Add ja in each languages.js .

What are the acceptance criteria?

@MattIPv4
Copy link
Contributor

👋 Hey @Ippey, would you mind updating the Copyright xxxx value in the files you've modified to be 2022?

I've also just merged #366 which added two new strings that your language pack will need 💙

Tyvm!

@Ippey
Copy link
Contributor Author

Ippey commented Jun 15, 2022

@MattIPv4 I've just modified it!

- templates.globalSections.performance.disableHtmlCaching
- templates.globalSections.performance.enableDisableHtmlCaching
@Ippey
Copy link
Contributor Author

Ippey commented Jun 15, 2022

I'm sorry. I added disableHtmlCaching, enableDisableHtmlCaching translation in ja. 🙏

@5idereal
Copy link
Contributor

Hi, could you take a look at the suggestions I made above?

@Ippey
Copy link
Contributor Author

Ippey commented Jun 15, 2022

Hi, @SiderealArt ! Does it mean that I have to modify year 20** to 2022 not only in the ja folder's files but also other languages files I modified?

@5idereal
Copy link
Contributor

Yes, as @MattIPv4 said earlier.

@Ippey
Copy link
Contributor Author

Ippey commented Jun 15, 2022

@MattIPv4 I also modified every languages.js.

Thank you, @SiderealArt !

@@ -34,4 +34,5 @@ export default {
ru: 'Russisch',
pl: 'Polnisch',
de: 'Deutsch',
ja: 'Japans',
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
ja: 'Japans',
ja: 'Japanisch',

@@ -34,4 +34,5 @@ export default {
ru: 'Russian',
pl: 'Polish',
de: 'German',
ja: 'Japans',
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
ja: 'Japans',
ja: 'Japanese',

@5idereal
Copy link
Contributor

I forgot to click comment. I meant these suggestions. Sorry for bothering you again. 😥

Copy link
Contributor

@MattIPv4 MattIPv4 left a comment

Choose a reason for hiding this comment

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

💙 Thank you for this translation -- a few quick thoughts:

@@ -34,4 +34,5 @@ export default {
ru: 'Russisch',
pl: 'Polnisch',
de: 'Deutsch',
ja: 'Japans',
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
ja: 'Japans',
ja: 'Japanisch',

@@ -34,4 +34,5 @@ export default {
ru: 'Russian',
pl: 'Polish',
de: 'German',
ja: 'Japans',
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
ja: 'Japans',
ja: 'Japanese',

Comment on lines 28 to 47
back: '戻る',
next: '次へ',
enable: '有効にする',
php: 'PHP',
ssl: 'SSL',
nginx: 'NGINX',
http: 'HTTP',
https: 'HTTPS',
letsEncrypt: 'Let\'s Encrypt',
python: 'Python',
wordPress: 'WordPress',
drupal: 'Drupal',
magento: 'Magento',
joomla: 'Joomla',
django: 'Django',
logging: 'ログ',
reverseProxy: 'リバースプロキシ',
reverseProxyLower: 'リバースプロキシ',
restrict: '制限',
path: 'パス',
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
back: '戻る',
next: '次へ',
enable: '有効にする',
php: 'PHP',
ssl: 'SSL',
nginx: 'NGINX',
http: 'HTTP',
https: 'HTTPS',
letsEncrypt: 'Let\'s Encrypt',
python: 'Python',
wordPress: 'WordPress',
drupal: 'Drupal',
magento: 'Magento',
joomla: 'Joomla',
django: 'Django',
logging: 'ログ',
reverseProxy: 'リバースプロキシ',
reverseProxyLower: 'リバースプロキシ',
restrict: '制限',
path: 'パス',
back: '戻る',
next: '次へ',
enable: '有効にする',
php: 'PHP',
ssl: 'SSL',
nginx: 'NGINX',
http: 'HTTP',
https: 'HTTPS',
letsEncrypt: 'Let\'s Encrypt',
python: 'Python',
wordPress: 'WordPress',
drupal: 'Drupal',
magento: 'Magento',
joomla: 'Joomla',
django: 'Django',
logging: 'ログ',
reverseProxy: 'リバースプロキシ',
reverseProxyLower: 'リバースプロキシ',
restrict: '制限',
path: 'パス',

Looks like some tabs ended up in here, should be four space indents :)

Comment on lines 28 to 36
en: 'English',
es: 'Spanish',
zhCN: 'Chinese (simplified)',
zhTW: 'Chinese (traditional)',
ptBR: 'Portuguese (brazilian)',
fr: 'French',
ru: 'Russian',
pl: 'Polish',
de: 'German',
Copy link
Contributor

Choose a reason for hiding this comment

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

These can be translated to Japanese :)


export default {
wantToContributeChanges: '👋 新機能や変更の提案や実装、翻訳をしてみませんか?',
getInvolvedOnGitHub: 'Get involved on GitHub',
Copy link
Contributor

Choose a reason for hiding this comment

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

Should this be translated?


export default {
disableHtmlCaching: 'HTML キャッシュの無効化',
enableDisableHtmlCaching: 'TML キャッシュを無効にする',
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
enableDisableHtmlCaching: 'TML キャッシュを無効にする',
enableDisableHtmlCaching: 'HTML キャッシュを無効にする',

import common from '../../common';

export default {
letsGoLive: 'Let\'s go live!',
Copy link
Contributor

Choose a reason for hiding this comment

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

Can this be translated?

export default {
letsGoLive: 'Let\'s go live!',
reloadNginxToLoadInYourNewConfiguration: `${common.nginx} をリロードして新しい設定を反映します:`,
goLive: 'Go live!',
Copy link
Contributor

Choose a reason for hiding this comment

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

Can this be translated?

@@ -34,4 +34,5 @@ export default {
ru: 'Rosyjski',
pl: 'Polski',
de: 'Niemiecki',
ja: 'Japonês',
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
ja: 'Japonês',
ja: 'Japoński',

@Ippey
Copy link
Contributor Author

Ippey commented Jun 15, 2022

@MattIPv4 @SiderealArt Thank you for your suggestions. I corrected them.

Copy link
Contributor

@MattIPv4 MattIPv4 left a comment

Choose a reason for hiding this comment

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

LGTM, thank you very much for contributing this! 💙

@MattIPv4 MattIPv4 merged commit 97b175a into digitalocean:master Jun 16, 2022
@MattIPv4
Copy link
Contributor

Hey, @Ippey - Thanks a ton for this sweet PR! 😄

Would you please shoot me an email when you get a chance?
mcowley at digitalocean dot com 🎉

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