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.

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

Translate Error Boundaries #81

Merged
merged 7 commits into from
Feb 13, 2019

Conversation

uenitty
Copy link
Contributor

@uenitty uenitty commented Feb 6, 2019

This PR is part of #4

Translation of Error Boundaries

Please review 🙏

@smikitky smikitky self-requested a review February 6, 2019 11:39
@smikitky
Copy link
Member

smikitky commented Feb 6, 2019

原文での表現にかかわらず error boundary と単数表記でお願いできますでしょうか。(refs, HoCs や components なども同様です) 細かいレビューは改めて行いたいと思います。

@uenitty uenitty force-pushed the feature/translate_error_boundaries branch from bf70fb6 to 48729cc Compare February 6, 2019 12:00
@uenitty
Copy link
Contributor Author

uenitty commented Feb 6, 2019

ありがとうございます。単数表記に統一致しました 🙇
よろしくお願い致します 🙏

@uenitty uenitty force-pushed the feature/translate_error_boundaries branch from 48729cc to 53c6e4e Compare February 8, 2019 12:04
@uenitty uenitty force-pushed the feature/translate_error_boundaries branch from 53c6e4e to d40581d Compare February 8, 2019 12:06
@uenitty
Copy link
Contributor Author

uenitty commented Feb 8, 2019

#4 (comment) に合わせてrebase & force pushしました 🙇

@tesseralis tesseralis mentioned this pull request Feb 9, 2019
90 tasks
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.

@uenitty 一点だけ不要な改行があったのでそこだけ削除お願いします!あとはOKだと思います! 👏


<img src="../images/docs/error-boundaries-stack-trace-line-numbers.png" style="max-width:100%" alt="Error caught by Error Boundary component with line numbers">

If you don’t use Create React App, you can add [this plugin](https://www.npmjs.com/package/babel-plugin-transform-react-jsx-source) manually to your Babel configuration. Note that it’s intended only for development and **must be disabled in production**.
Create React App を使用しない場合は、[このプラグイン](https://www.npmjs.com/package/babel-plugin-transform-react-jsx-source)を手動で Babel の設定に追加してください。
ただし、この機能は開発専用であり、**本番では必ず無効化しなければならない**ことに注意してください。
Copy link
Member

Choose a reason for hiding this comment

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

原文にはない改行があるので、削除お願いします 🙏

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.

@uenitty 対応ありがとうございます!

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.

微修正の提案のみです。

タイトル以外の本文中に出てくる "Error Boundary" は小文字に統一(文頭であっても)でお願いします。ref や props なども小文字にになっていますので。

content/docs/error-boundaries.md Outdated Show resolved Hide resolved
content/docs/error-boundaries.md Outdated Show resolved Hide resolved
> * Errors thrown in the error boundary itself (rather than its children)
> * イベントハンドラ([詳細](#how-about-event-handlers))
> * 非同期コード(例:`setTimeout` や `requestAnimationFrame` のコールバック)
> * サーバーサイドレンダリング
Copy link
Contributor Author

Choose a reason for hiding this comment

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

ここだけレンダリングを残しました

@smikitky smikitky merged commit a32e2d3 into reactjs:master Feb 13, 2019
@smikitky
Copy link
Member

ありがとうございました!

@uenitty
Copy link
Contributor Author

uenitty commented Feb 13, 2019

こちらこそご丁寧にレビューしていただきありがとうございました!
実は初OSSコントリビュートでしたが無事マージされてホッとしております 🤗

@uenitty uenitty deleted the feature/translate_error_boundaries branch February 13, 2019 03:35
@koba04
Copy link
Member

koba04 commented Feb 13, 2019

@uenitty おめでとうございます 🎉🎉🎉

@uenitty uenitty mentioned this pull request Mar 3, 2019
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