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.

Sign up for GitHub

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

Jump to bottom

Translated implementation notes #178

Merged
merged 43 commits into from
Aug 6, 2019
Merged

Translated implementation notes #178

merged 43 commits into from
Aug 6, 2019

Conversation

y-yeah
Copy link
Contributor

@y-yeah y-yeah commented Mar 22, 2019

No description provided.

y-yeah added 15 commits March 8, 2019 09:36
@y-yeah
translating the intro part of implementation notes
dd3ca4b
@y-yeah
Merge branch 'master' of https://github.com/reactjs/ja.reactjs.org
68f5768
@y-yeah
completed mounting as a recursive process in implementation notes
78a7dd8
@y-yeah
working on mounting host elements in implementation notes
e8b7f61
@y-yeah
completed internal instance intro in implementation notes
9911bc6
@y-yeah
translated unmounting in implementation notes
d432f2f
@y-yeah
Merge branch 'master' of https://github.com/reactjs/ja.reactjs.org
c2ba930
@y-yeah
translated updating in implementation notes
8dfcf04
@y-yeah
translated updating composite components in implementation notes
7a3242c
@y-yeah
translated updating host components in implementation notes
79d52db
@y-yeah
Merge branch 'master' of https://github.com/reactjs/ja.reactjs.org
8dce8aa
@y-yeah
translated what we left out in implementation notes
cd1f87a
@y-yeah
translated jumping into the Code in implmentation notes
07e3542
@y-yeah
completed translation of implementation notes
f45f4c4
@y-yeah
ran yarn check-all on implementation notes
754437f
@netlify
Copy link

netlify bot commented Mar 22, 2019 •
edited
Loading

Deploy preview for ja-reactjs ready!

Built with commit 15c3811

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

@koba04
Copy link
Member

koba04 commented Mar 22, 2019

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

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

@y-yeah
Copy link
Contributor Author

y-yeah commented Mar 24, 2019

@koba04

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

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

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

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

@koba04
Copy link
Member

koba04 commented Mar 25, 2019

@y-yeah

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

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

@y-yeah
Copy link
Contributor Author

y-yeah commented Mar 26, 2019

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

@koba04 koba04 self-requested a review March 26, 2019 15:51
koba04
koba04 requested changes Mar 27, 2019
View reviewed changes
Copy link
Member

@koba04 koba04 left a comment

Choose a reason for hiding this comment

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

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

@koba04
Copy link
Member

koba04 commented May 7, 2019

@potato4d @smikitky Please review this PR 🙏

@potato4d potato4d self-requested a review May 7, 2019 09:47
@potato4d
Copy link
Member

potato4d commented May 7, 2019

@koba04
i will check it within few days 🙇

smikitky
smikitky requested changes May 16, 2019
View reviewed changes
Copy link
Member

@smikitky smikitky left a comment

Choose a reason for hiding this comment

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

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

y-yeah and others added 10 commits May 18, 2019 11:31
@y-yeah @smikitky
Update content/docs/implementation-notes.md
0562743 
Co-Authored-By: Soichiro Miki <[email protected]>
@y-yeah @smikitky
Update content/docs/implementation-notes.md
add9a9a 
Co-Authored-By: Soichiro Miki <[email protected]>
@y-yeah @smikitky
Update content/docs/implementation-notes.md
945c8cd 
Co-Authored-By: Soichiro Miki <[email protected]>
@y-yeah @smikitky
Update content/docs/implementation-notes.md
850a28f 
Co-Authored-By: Soichiro Miki <[email protected]>
@y-yeah @smikitky
Update content/docs/implementation-notes.md
a00c79e 
Co-Authored-By: Soichiro Miki <[email protected]>
@y-yeah @smikitky
Update content/docs/implementation-notes.md
07e4f31 
Co-Authored-By: Soichiro Miki <[email protected]>
@y-yeah @smikitky
Update content/docs/implementation-notes.md
4d7b1ed 
Co-Authored-By: Soichiro Miki <[email protected]>
@y-yeah @smikitky
Update content/docs/implementation-notes.md
da9419b 
Co-Authored-By: Soichiro Miki <[email protected]>
@y-yeah @smikitky
Update content/docs/implementation-notes.md
c381dc4 
Co-Authored-By: Soichiro Miki <[email protected]>
@y-yeah @smikitky
Update content/docs/implementation-notes.md
8922cae 
Co-Authored-By: Soichiro Miki <[email protected]>
@y-yeah
Copy link
Contributor Author

y-yeah commented May 18, 2019

@koba04 @smikitky

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

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

@smikitky
Copy link
Member

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

@y-yeah @smikitky
Apply suggestions from code review
26499a6 
大変遅くなりましたが、修正点確認しました。

Co-Authored-By: Soichiro Miki <[email protected]>
@y-yeah
Copy link
Contributor Author

y-yeah commented Jun 3, 2019

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

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

@smikitky
Copy link
Member

smikitky commented Jun 4, 2019

@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.

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

smikitky and others added 4 commits June 25, 2019 11:55
@smikitky
Merge branch 'master' into implementation_notes
0ff346d
@smikitky
Merge branch 'master' into implementation_notes
2913f69
@y-yeah
Merge branch 'master' into implementation_notes
17e29bf
@y-yeah @smikitky
Apply suggestions from code review
47f8884 
すみません、コメントがあるのを見逃していました。
遅くなってしまいましたが、確認お願いします。

Co-Authored-By: Soichiro Miki <[email protected]>
potato4d
potato4d approved these changes Aug 6, 2019
View reviewed changes
Copy link
Member

@potato4d potato4d left a comment

Choose a reason for hiding this comment

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

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

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

@potato4d potato4d merged commit d56a69f into reactjs:master Aug 6, 2019
@potato4d potato4d mentioned this pull request Aug 6, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@smikitky smikitky smikitky requested changes

@koba04 koba04 koba04 approved these changes

@potato4d potato4d potato4d approved these changes

@hg-pyun hg-pyun hg-pyun approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@y-yeah @koba04 @potato4d @smikitky @hg-pyun