Skip to content

upgrade messages_ja.json - #3

Draft
Martsings wants to merge 15 commits into
masterfrom
8898-messages_ja.json
Draft

upgrade messages_ja.json#3
Martsings wants to merge 15 commits into
masterfrom
8898-messages_ja.json

Conversation

@Martsings

Copy link
Copy Markdown
Member

Changes

upgarde messages_ja.json

@kobaji kobaji changed the title upgrade messages_ja.json wip:upgrade messages_ja.json Oct 20, 2020
@kobaji kobaji changed the title wip:upgrade messages_ja.json upgrade messages_ja.json Oct 20, 2020
@kobaji
kobaji marked this pull request as draft October 22, 2020 01:18

@kobaji kobaji left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

リソースの種類(Namespace, Task等)は、大文字始まりで統一した方が良いと思いました。英語が小文字始まりだとしても。

確かに日本語の表記で普通大文字始まりですね。こちらは修正します。

→修正を確認しました。この点に関して追加指摘はありません。
今後コミットが発生した場合、必要に応じて差分の英単語も確認するようにします。

Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json Outdated

@kobaji kobaji left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

submitできていなかったです。すみません。

Comment thread src/nls/messages_ja.json
Comment thread src/nls/messages_ja.json
Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json
Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json Outdated

@kobaji kobaji left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

submit

Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json
Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json
Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json Outdated

@kobaji kobaji left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

追加ですみません。句点「。」をなくしましょう。

Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json Outdated
@kobaji

kobaji commented Oct 27, 2020

Copy link
Copy Markdown
Member

本日のWebUIによるレビュー時、小林のAIになっていたものです。
句点をなくすかどうかの指摘は追加ですみません。句点「。」をなくしましょう。というコメントに記載しています。

作業履歴

  1. https://github.com/fujitsu-cdf/dashboard/blob/8898-messages_ja.json/src/nls/messages_ja.json をコピペ
  2. ドキュメント内を「。」でGREP
  3. 1文のみになっているものについて、句点を削除するコメントを残した。

Comment thread src/nls/messages_ja.json Outdated
@yamaken1343
yamaken1343 self-requested a review October 29, 2020 01:35

@yamaken1343 yamaken1343 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm

Comment thread src/nls/messages_ja.json Outdated
Comment thread src/nls/messages_ja.json Outdated
@kobaji

kobaji commented Oct 29, 2020

Copy link
Copy Markdown
Member

すべてResolveしました。作成、修正お疲れ様でした!

load messages_ja.json file to translations file branch
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