-
Notifications
You must be signed in to change notification settings - Fork 177
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 useExternalStore reference #619
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
レビューを行いましたので対応をお願いします。
ChatGPT を信用せず、全文丁寧に添削する方向でお願いします。訳抜けやタグ抜け、明らかな誤訳が結構放置されています。またローカルで dev server を立ち上げて、そもそもちゃんとページの表示ができているかの確認をお願いします。
@smikitky
お手数おかけしますが、再度レビューお願いします。 |
Size changes📦 Next.js Bundle Analysis for react-devThis analysis was generated by the Next.js Bundle Analysis action. 🤖 This PR introduced no changes to the JavaScript bundle! 🙌 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ありがとうございます。ほぼ OK なのですが幾つか私の方で見逃しがあったので対応をお願いします:pray:
Co-authored-by: Soichiro Miki <smiki-tky@umin.ac.jp>
レビュー、ありがとうございます! 71ea816 で code suggestion を squash して push -f して反映しました。 よろしくお願いします🙏 |
ありがとうございました! 2 名が approve してからのマージになりますのでしばらくお待ちください。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 ありがとうございます!!!
useExternalStore
の翻訳ですやったこと