Skip to content

Translate "cache" #696

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 41 commits into from
Oct 12, 2023
Merged

Translate "cache" #696

merged 41 commits into from
Oct 12, 2023

Conversation

locol23
Copy link
Contributor

@locol23 locol23 commented Sep 29, 2023

"cache" の翻訳になります。

@github-actions
Copy link

github-actions bot commented Oct 5, 2023

Size changes

📦 Next.js Bundle Analysis for react-dev

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

@locol23 locol23 marked this pull request as ready for review October 5, 2023 18:46
@locol23
Copy link
Contributor Author

locol23 commented Oct 5, 2023

@smikitky
お待たせしてすみません。お手隙の際にレビューをお願いできないでしょうか 🙏

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.

レビューをしましたので確認をよろしくお願いします。


### `cache(fn)` {/*cache*/}

Call `cache` outside of any components to create a version of the function with caching.
キャッシュを持つバージョンの関数を作成するために、コンポーネントの外部で `cache` を呼び出します。
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
キャッシュを持つバージョンの関数を作成するために、コンポーネントの外部で `cache` を呼び出します
コンポーネントの外部で `cache` を呼び出して、キャッシュが有効化されたバージョンの関数を作成します

他のリファレンス記事との表記統一です

Copy link
Contributor Author

Choose a reason for hiding this comment

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

横断的に見ていないとわからない部分のため、表記統一ありがとうございます 🙏

Co-authored-by: Soichiro Miki <smiki-tky@umin.ac.jp>
locol23 and others added 2 commits October 11, 2023 23:34
Co-authored-by: Soichiro Miki <smiki-tky@umin.ac.jp>
Co-authored-by: Soichiro Miki <smiki-tky@umin.ac.jp>
Co-authored-by: Soichiro Miki <smiki-tky@umin.ac.jp>
locol23 and others added 5 commits October 11, 2023 23:38
Co-authored-by: Soichiro Miki <smiki-tky@umin.ac.jp>
Co-authored-by: Soichiro Miki <smiki-tky@umin.ac.jp>
Co-authored-by: Soichiro Miki <smiki-tky@umin.ac.jp>
Co-authored-by: Soichiro Miki <smiki-tky@umin.ac.jp>
Co-authored-by: Soichiro Miki <smiki-tky@umin.ac.jp>
Co-authored-by: Soichiro Miki <smiki-tky@umin.ac.jp>
Co-authored-by: Soichiro Miki <smiki-tky@umin.ac.jp>
Co-authored-by: Soichiro Miki <smiki-tky@umin.ac.jp>
Co-authored-by: Soichiro Miki <smiki-tky@umin.ac.jp>
locol23 and others added 2 commits October 12, 2023 09:56
Co-authored-by: Soichiro Miki <smiki-tky@umin.ac.jp>
Co-authored-by: Soichiro Miki <smiki-tky@umin.ac.jp>
Co-authored-by: Soichiro Miki <smiki-tky@umin.ac.jp>
Co-authored-by: Soichiro Miki <smiki-tky@umin.ac.jp>
@smikitky
Copy link
Member

初期状態で隠れているレビューコメントを全部表示するのを忘れているようで、幾つかのレビューコメントに対応されていないようですので対応をお願いします。

またこちらの点についてもよろしくお願いします。

@locol23
Copy link
Contributor Author

locol23 commented Oct 12, 2023

@smikitky すみません。まだ確認途中でした 🙏 残りのレビューコメントも確認します 🙇 また、バッチについて承知しました。対応します。

Co-authored-by: Soichiro Miki <smiki-tky@umin.ac.jp>
@locol23
Copy link
Contributor Author

locol23 commented Oct 12, 2023

@smikitky 残っていたレビューコメントの反映と、提案し、確認して頂いた 2 点の修正が完了しましたので、お手すきでご確認をお願いできないでしょうか 🙇

■ 確認して頂いた 2 点

  1. Translate "cache" #696 (comment)
  2. Translate "cache" #696 (comment)

上記 2 点を修正したコミット

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.

ありがとうございました!
一部微修正させていただきました。

@locol23
Copy link
Contributor Author

locol23 commented Oct 12, 2023

@smikitky こちらこそ、丁寧に見て頂いてありがとうございました!微修正もありがとうございます 🙏

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.

👍 ありがとうございます!

@koba04 koba04 merged commit 2844162 into reactjs:main Oct 12, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants