Skip to content

Conversation

@Martsings
Copy link
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
Copy link
Member

@kobaji kobaji left a comment

Choose a reason for hiding this comment

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

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

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

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

Copy link
Member

@kobaji kobaji left a comment

Choose a reason for hiding this comment

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

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

Copy link
Member

@kobaji kobaji left a comment

Choose a reason for hiding this comment

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

submit

Copy link
Member

@kobaji kobaji left a comment

Choose a reason for hiding this comment

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

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

@kobaji
Copy link
Member

kobaji commented Oct 27, 2020

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

作業履歴

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

@yamaken1343 yamaken1343 self-requested a review October 29, 2020 01:35
Copy link
Member

@yamaken1343 yamaken1343 left a comment

Choose a reason for hiding this comment

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

lgtm

@kobaji
Copy link
Member

kobaji commented Oct 29, 2020

すべて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.

4 participants