-
-
Notifications
You must be signed in to change notification settings - Fork 473
docs: remove duplicated English content and improve translation clarity in useActionState docs #1194
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
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Skipped Deployment
|
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.
안녕하세요, PR 보내주셔서 감사합니다.
다만, 영문 내용이 최신 버전이라, 영문 내용을 반영해서 새로 업데이트 해주실 수 있으실까요?
그리고 체크박스도 한번씩 체크 부탁드리겠습니다.
@lumirlumir 해당 피드백 반영하여 최신 React 영문 내용 반영한 Commit 및 PR 내용 업데이트 하였습니다. |
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.
리뷰 확인 부탁드리겠습니다!
2. `<form>` 컴포넌트의 `action` Prop이나 폼 내부 `<button>` 컴포넌트의 `formAction` Prop에 전달할 수 있는 새 액션입니다. | ||
3. 폼 액션이 대기 중인지 여부를 알려주는 `isPending` 플래그입니다. | ||
1. 현재 State입니다. 첫 렌더링 시에는 `initialState`와 일치하며, 액션이 실행된 후에는 해당 액션이 반환한 값과 일치합니다. | ||
2. `<form>` 컴포넌트의 `action` Prop이나, 폼 내부 `<button>` 컴포넌트의 `formAction` Prop에 전달할 수 있는 새 액션입니다. 이 액션은 [startTransition](/reference/react/startTransition) 내에서 수동으로 호출할 수도 있습니다. |
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.
2. `<form>` 컴포넌트의 `action` Prop이나, 폼 내부 `<button>` 컴포넌트의 `formAction` Prop에 전달할 수 있는 새 액션입니다. 이 액션은 [startTransition](/reference/react/startTransition) 내에서 수동으로 호출할 수도 있습니다. | |
2. `form` 컴포넌트의 `action` Prop이나, 폼 내부 `button` 컴포넌트의 `formAction` Prop에 전달할 수 있는 새 액션입니다. 이 액션은 [`startTransition`](/reference/react/startTransition) 내에서 수동으로 호출할 수도 있습니다. |
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.
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!
변경사항 1
useActionState 섹션의 번역 문서에서 중복된 영어 원문(항목 4~6) 을 제거했습니다.
항목 1, 2, 3에서 이미 한국어로 번역된 설명이 충분히 제공되고 있음에도 불구하고, 바로 아래에 영어 원문(항목 4, 5, 6)이 다시 나열되어 있어 독자의 혼란을 유발할 수 있습니다.
해당 중복 내용은 문서의 가독성을 떨어뜨리고, 다른 번역 문서들과 스타일의 일관성도 맞지 않습니다.
React 공식 문서의 번역 지침에서도 중복 없이 자연스러운 번역 제공을 권장하고 있습니다.
따라서, 해당 내용 수정을 통해 문서의 명료성을 향상 시키고 독자의 혼란을 방지하여 전체 번역 문서의 스타일 및 품질 개선을 기대합니다.
변경사항 2
useActionState
훅에 대한 번역을 최신 영어 원문에 맞춰 업데이트하고, 표현을 보다 자연스럽고 명확하게 다듬었습니다.기존 번역은 기본적인 의미 전달에는 문제가 없었지만, 일부 문장이 다소 부자연스럽고 기술적인 뉘앙스가 정확하지 않은 부분이 있었습니다.
또한, 최신 React 공식 문서 영문 내용을 기준으로 정확한 의미 전달과 문체 개선이 필요하다고 판단하여 수정했습니다.
React 영문 docs
감사합니다.
필수 확인 사항
textlint
가이드Textlint Guide선택 확인 사항