Translated implementation notes by y-yeah · Pull Request #178 · reactjs/ja.react.dev

y-yeah · 2019-03-22T01:46:14Z

No description provided.

netlify · 2019-03-22T01:51:33Z

Deploy preview for ja-reactjs ready!

Built with commit 15c3811

https://deploy-preview-178--ja-reactjs.netlify.com

koba04 · 2019-03-22T12:12:22Z

@y-yeah
翻訳お疲れ様です！
レビュー前に2点ほど確認お願いします 🙇

対象ページ以外にも差分が出ているようなので解消お願いします
可能な限り、原文と改行の位置などをあわせて行毎の対応関係がわかるようにお願いします。（レビューや今後の原文の更新に対応しやすくなるため）

y-yeah · 2019-03-24T15:20:20Z

@koba04

@y-yeah
翻訳お疲れ様です！
レビュー前に2点ほど確認お願いします 🙇
対象ページ以外にも差分が出ているようなので解消お願いします
可能な限り、原文と改行の位置などをあわせて行毎の対応関係がわかるようにお願いします。（レビューや今後の原文の更新に対応しやすくなるため）

差分について質問があります！
PR前に全体をリンティングにかけたところ、エラーが出ていたため直すべきと判断しました。その部分が今回の他ファイルでの差分になります。
この２箇所のエラーは他翻訳者の担当部分ということで、今回はエラーのまま見送ればよいのでしょうか。

改行に関しては承知しました！

koba04 · 2019-03-25T04:33:53Z

@y-yeah

この２箇所のエラーは他翻訳者の担当部分ということで、今回はエラーのまま見送ればよいのでしょうか。

CIのチェックではエラーにはなっていないようなのでそのままでもOKです。
修正する場合は別PRにしてもらえるとそれだけをmerge出来るのでありがたいです。
（今回の場合は、reactjs.orgのリポジトリに対して修正する方がよさそう？）

…to implementation_notes

y-yeah · 2019-03-26T11:27:57Z

@koba04
ひとまず、ご指摘いただいた箇所を修正しました！

koba04

@y-yeah お疲れ様です！いくつかコメントしましたので確認お願いします 🙇

Co-Authored-By: y-yeah <39003567+y-yeah@users.noreply.github.com>

added missing translation

koba04 · 2019-05-07T09:46:39Z

@potato4d @smikitky Please review this PR 🙏

potato4d · 2019-05-07T09:48:19Z

@koba04
i will check it within few days 🙇

smikitky

遅くなり申し訳ありません。大部分がスタイルに関わる部分の指摘です。よろしくお願いします。

Co-Authored-By: Soichiro Miki <smiki-tky@umin.ac.jp>

y-yeah · 2019-05-18T02:35:41Z

@koba04 @smikitky

遅くなり申し訳ありません。大部分がスタイルに関わる部分の指摘です。よろしくお願いします。

ご指摘いただいた部分すべて確認いたしました！

smikitky · 2019-05-20T04:28:25Z

@y-yeah "19 hidden conversations"で非表示になっている部分についてまだ対応いただけていないようですので確認をお願いします。
あと何十もコミットが作られると見づらくなるので、可能ならバッチコミットを使ってください。画面上の方の「Files Changed」の方からレビューコメントの対応をしていただければWeb画面上でも1コミットでレビュー対応ができます。

大変遅くなりましたが、修正点確認しました。 Co-Authored-By: Soichiro Miki <smiki-tky@umin.ac.jp>

y-yeah · 2019-06-03T05:39:55Z

@y-yeah "19 hidden conversations"で非表示になっている部分についてまだ対応いただけていないようですので確認をお願いします。
あと何十もコミットが作られると見づらくなるので、可能ならバッチコミットを使ってください。画面上の方の「Files Changed」の方からレビューコメントの対応をしていただければWeb画面上でも1コミットでレビュー対応ができます。

バッチコミットについて教えていただきありがとうございます。全て対応いたしました。

smikitky · 2019-06-04T04:28:39Z

@y-yeah なんども申し訳ありません、1か所だけsuggestionが残っているので確認の上で対応をお願いします。

Next, we can look at the rendered element’s type. If the type has not changed since the last render, the component below can also be updated in place.

の部分、「下記のコンポーネント」となっていますが、これは「下記のコンポーネント」ではなく、「配下のコンポーネント」「子孫コンポーネント」などとすべきだと思います。

すみません、コメントがあるのを見逃していました。遅くなってしまいましたが、確認お願いします。 Co-Authored-By: Soichiro Miki <smiki-tky@umin.ac.jp>

potato4d

これどうするか悩んでたんですが、あとやるべきは @smikitky が出している Change Request 一点だけだと思うので、一旦マージしてこっちで Pull Request 別途出しちゃいますね。

@y-yeah
ありがとうございました！

y-yeah added 15 commits March 8, 2019 09:36

translating the intro part of implementation notes
dd3ca4b

Merge branch 'master' ofhttps://github.com/reactjs/ja.reactjs.org
68f5768

completed mounting as a recursive process in implementation notes
78a7dd8

working on mounting host elements in implementation notes
e8b7f61

completed internal instance intro in implementation notes
9911bc6

translated unmounting in implementation notes
d432f2f

Merge branch 'master' ofhttps://github.com/reactjs/ja.reactjs.org
c2ba930

translated updating in implementation notes
8dfcf04

translated updating composite components in implementation notes
7a3242c

translated updating host components in implementation notes
79d52db

Merge branch 'master' ofhttps://github.com/reactjs/ja.reactjs.org
8dce8aa

translated what we left out in implementation notes
cd1f87a

translated jumping into the Code in implmentation notes
07e3542

completed translation of implementation notes
f45f4c4

ran yarn check-all on implementation notes
754437f

y-yeah added 5 commits March 26, 2019 20:00

Merge branch 'master' ofhttps://github.com/reactjs/ja.reactjs.orgin…
82fc439
…to implementation_notes

deleted modification in the other files than implementation notes
dd88317

fixed lines in Implementation notes
866d392

fixed mistakes in the second-to-last paragraph in Implementation notes
c79bc09

fixed line numbers in Implementation notes
e3c7777

koba04 self-requested a review March 26, 2019 15:51

koba04 requested changes Mar 27, 2019
View reviewed changes

Comment threadcontent/community/conferences.md Outdated
Comment threadcontent/docs/implementation-notes.md Outdated
Comment threadcontent/docs/implementation-notes.md Outdated

koba04and others added 3 commits March 28, 2019 10:03

Update content/community/conferences.md
09a3951
Co-Authored-By: y-yeah <39003567+y-yeah@users.noreply.github.com>

Update content/docs/implementation-notes.md
62d82a9
Co-Authored-By: y-yeah <39003567+y-yeah@users.noreply.github.com>

Update on Implementation Notes
e9db403
added missing translation

potato4d self-requested a review May 7, 2019 09:47

smikitky requested changes May 16, 2019
View reviewed changes

y-yeahand others added 10 commits May 18, 2019 11:31

Update content/docs/implementation-notes.md
0562743
Co-Authored-By: Soichiro Miki <smiki-tky@umin.ac.jp>

Update content/docs/implementation-notes.md
add9a9a
Co-Authored-By: Soichiro Miki <smiki-tky@umin.ac.jp>

Update content/docs/implementation-notes.md
945c8cd
Co-Authored-By: Soichiro Miki <smiki-tky@umin.ac.jp>

Update content/docs/implementation-notes.md
850a28f
Co-Authored-By: Soichiro Miki <smiki-tky@umin.ac.jp>

Update content/docs/implementation-notes.md
a00c79e
Co-Authored-By: Soichiro Miki <smiki-tky@umin.ac.jp>

Update content/docs/implementation-notes.md
07e4f31
Co-Authored-By: Soichiro Miki <smiki-tky@umin.ac.jp>

Update content/docs/implementation-notes.md
4d7b1ed
Co-Authored-By: Soichiro Miki <smiki-tky@umin.ac.jp>

Update content/docs/implementation-notes.md
da9419b
Co-Authored-By: Soichiro Miki <smiki-tky@umin.ac.jp>

Update content/docs/implementation-notes.md
c381dc4
Co-Authored-By: Soichiro Miki <smiki-tky@umin.ac.jp>

Update content/docs/implementation-notes.md
8922cae
Co-Authored-By: Soichiro Miki <smiki-tky@umin.ac.jp>

Merge branch 'master' into implementation_notes
ab2abed

Apply suggestions from code review
26499a6
大変遅くなりましたが、修正点確認しました。 Co-Authored-By: Soichiro Miki <smiki-tky@umin.ac.jp>

Merge branch 'master' into implementation_notes
5c5100e

hg-pyun approved these changes Jun 15, 2019
View reviewed changes

smikitkyand others added 4 commits June 25, 2019 11:55

Merge branch 'master' into implementation_notes
0ff346d

Merge branch 'master' into implementation_notes
2913f69

Merge branch 'master' into implementation_notes
17e29bf

Apply suggestions from code review
47f8884
すみません、コメントがあるのを見逃していました。遅くなってしまいましたが、確認お願いします。 Co-Authored-By: Soichiro Miki <smiki-tky@umin.ac.jp>

potato4d approved these changes Aug 6, 2019
View reviewed changes

Merge branch 'master' into implementation_notes
15c3811

potato4d merged commit d56a69f into reactjs:masterAug 6, 2019

potato4d mentioned this pull request Aug 6, 2019
chore: Fix implementation notes wording and text weight #232
Merged

Uh oh!

Conversation

y-yeah commented Mar 22, 2019

Uh oh!

netlifyBot commented Mar 22, 2019• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

koba04 commented Mar 22, 2019

Uh oh!

y-yeah commented Mar 24, 2019

Uh oh!

koba04 commented Mar 25, 2019

Uh oh!

y-yeah commented Mar 26, 2019

Uh oh!

koba04 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

koba04 commented May 7, 2019

Uh oh!

potato4d commented May 7, 2019

Uh oh!

smikitky left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

y-yeah commented May 18, 2019

Uh oh!

smikitky commented May 20, 2019

Uh oh!

y-yeah commented Jun 3, 2019

Uh oh!

smikitky commented Jun 4, 2019

Uh oh!

potato4d left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

netlifyBot commented Mar 22, 2019•
edited
Loading