-
-
Notifications
You must be signed in to change notification settings - Fork 526
docs: update synchronizing-with-effects.md
#1393
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
docs: update synchronizing-with-effects.md
#1393
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. 1 Skipped Deployment
|
Size changesDetails📦 Next.js Bundle Analysis for undefinedThis analysis was generated by the Next.js Bundle Analysis action. 🤖 This PR introduced no changes to the JavaScript bundle! 🙌 |
synchronizing-with-effects.md
Co-authored-by: 루밀LuMir <rpfos@naver.com>
lumirlumir
left a comment
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.
LGTM, 기여 감사합니다!
|
참고로, |
|
해당 부분 인지하고 있습니다, 감사합니다..! |
close #1392
synchronizing-with-effects.md 수정 및 추가 번역
원문과 다른 부분을 수정하고, 번역이 빠진 부분을 추가 번역하였습니다.
추가로 cacheSignal.md 파일에 textlint 문제가 생겨 함께 개선하였습니다.
필수 확인 사항
textlint가이드Textlint Guide선택 확인 사항