Skip to content
This repository has been archived by the owner on Apr 12, 2023. It is now read-only.

英語版翻訳の改善 #942

Merged
merged 1 commit into from
Jun 16, 2022
Merged

Conversation

kasmide
Copy link
Contributor

@kasmide kasmide commented Mar 26, 2022

Issue 番号 / Issue ID

目的 / Purpose

  • 英語版の誤訳を修正する
  • 英語版の文章をより分かりやすいものにする

変更内容 / Changes

  • e3e1514 文意が原文と大きく異なる翻訳の修正
  • f956e61 表現の変更や軽微なミスの修正

破壊的変更をもたらしますか / Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request の種類 / Pull Request type

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

確認事項 / What to check

  • 文字数の変化で表示崩れが起きていないかの確認?

その他 / Other information

  • もし f956e61 を取り込むのが難しそうであれば、誤訳の修正 (e3e1514) の方だけをマージして頂いても構いません。

レビュー等よろしくお願いします


Internal IDs:

@@ -625,15 +625,15 @@
<comment>同意して陽性登録する</comment>
</data>
<data name="SubmitConsentPageTitle1" xml:space="preserve">
<value>Consent to positive registration</value>
<value>Consent to register as positive</value>
Copy link
Collaborator

Choose a reason for hiding this comment

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

こちら開発チームから、この部分、こうした方がさらに良いのではないかというご提案です。

Suggested change
<value>Consent to register as positive</value>
<value>Consent to register a positive test result</value>

Copy link
Contributor Author

Choose a reason for hiding this comment

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

提案ありがとうございます。テキストが表示される箇所が添付画像ヘッダー部ですので、あまり長くしてしまうと見切れてしまうのではないかと思ったのですがその点は大丈夫でしょうか。Screenshot_20220328-171032~2.png

Copy link
Collaborator

Choose a reason for hiding this comment

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

ご指摘の通り、文字数が限られる領域です。
いただいている案で一度取り込み進行して、スケジュールへの影響を見ながら(サジェスチョンしている内容について)画面チェックなどしようと言うことになっています(その際はご意見伺えれば幸いです。

Copy link
Collaborator

@keiji keiji left a comment

Choose a reason for hiding this comment

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

いただいた翻訳、取り込みの方向で進んでいます。
本当にありがとうございます!

一点だけ開発チームからサジェスチョンをコメントしましたので、ご確認いただけるとありがたいです。

@keiji
Copy link
Collaborator

keiji commented Mar 28, 2022

いただいた変更については、develop向けに加えてv2.0.0向けのリリースブランチであるrelease/v2.0.0にも取り込む方針です。
v2.0.0向けにコミットのcherry-pickとPull Requestの作成はぼくの方でやる予定です(@kasmide さんのお名前はコミットログのAuthorとして残ります)。
Pull Requestを作ったら @kasmide さんにメンションします。または、ご自身でrelease/v2.0.0宛てのPull Requestを作ってもいいよという場合は、そちらも歓迎いたします。よろしくお願いいたします。

@kasmide
Copy link
Contributor Author

kasmide commented Mar 28, 2022

取り込んでいただく方針になっているとのことで本当に嬉しいです。release/v2.0.0 への Pull Request の作成は @keiji さんにお任せしますのでまた何かあればよろしくお願いします

@keiji
Copy link
Collaborator

keiji commented Mar 28, 2022

こちらこそありがとうございます!
Pull Requestについては #944 に作成しました。こちら(develop向けとは別に取り込み予定です)。

@keiji keiji self-requested a review June 13, 2022 01:52
@keiji
Copy link
Collaborator

keiji commented Jun 13, 2022

@kasmide
おつかれさまです。いただいたPull Requestについてはv2.0.0で一部は取り込まれているのですが、v2.1.0でこのPull Requestをマージするために開発チームが作業を進めています。

リソースがコンフリクトしてしまっているので、解決をお願いできないでしょうか。

または、一旦このPull Requestをfeatureブランチに取り込んだ後、コンフリクトの解決はぼくの方で作業して、結果を見ていただくこともできます。

@kasmide
Copy link
Contributor Author

kasmide commented Jun 14, 2022

返信が遅くなってしまいすみません。また、この PR の取り込みに向けて作業して頂いているとのことで本当にありがとうございます。現時点の develop (1f95e85) に rebase し直しましたのでご確認のほどお願いします。

Copy link
Collaborator

@keiji keiji left a comment

Choose a reason for hiding this comment

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

LTGM!
ありがとうございます。開発チームに引き継ぎます!

@cocoa-dev003
Copy link
Contributor

ありがとうございます!
問題ありませんでしたので、こちらでマージさせていただきます。

@cocoa-dev003 cocoa-dev003 merged commit 559046b into cocoa-mhlw:develop Jun 16, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

英語版の誤訳について
3 participants