Skip to content

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

Merged
merged 3 commits into from
May 14, 2025

Conversation

eunwoo-levi
Copy link
Contributor

@eunwoo-levi eunwoo-levi commented May 14, 2025

변경사항 1

useActionState 섹션의 번역 문서에서 중복된 영어 원문(항목 4~6) 을 제거했습니다.

항목 1, 2, 3에서 이미 한국어로 번역된 설명이 충분히 제공되고 있음에도 불구하고, 바로 아래에 영어 원문(항목 4, 5, 6)이 다시 나열되어 있어 독자의 혼란을 유발할 수 있습니다.

해당 중복 내용은 문서의 가독성을 떨어뜨리고, 다른 번역 문서들과 스타일의 일관성도 맞지 않습니다.

React 공식 문서의 번역 지침에서도 중복 없이 자연스러운 번역 제공을 권장하고 있습니다.

image

  • React 공식 문서 useActionState 섹션 PR 부분

따라서, 해당 내용 수정을 통해 문서의 명료성을 향상 시키고 독자의 혼란을 방지하여 전체 번역 문서의 스타일 및 품질 개선을 기대합니다.

변경사항 2

useActionState 훅에 대한 번역을 최신 영어 원문에 맞춰 업데이트하고, 표현을 보다 자연스럽고 명확하게 다듬었습니다.

기존 번역은 기본적인 의미 전달에는 문제가 없었지만, 일부 문장이 다소 부자연스럽고 기술적인 뉘앙스가 정확하지 않은 부분이 있었습니다.

또한, 최신 React 공식 문서 영문 내용을 기준으로 정확한 의미 전달과 문체 개선이 필요하다고 판단하여 수정했습니다.

image
React 영문 docs

감사합니다.

필수 확인 사항

선택 확인 사항

  • 번역 초안 작성Draft Translation
  • 리뷰 반영Resolve Reviews

Copy link

vercel bot commented May 14, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
ko-legacy-reactjs-org ⬜️ Ignored (Inspect) Visit Preview May 14, 2025 1:12pm

Copy link
Collaborator

@lumirlumir lumirlumir left a comment

Choose a reason for hiding this comment

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

안녕하세요, PR 보내주셔서 감사합니다.

다만, 영문 내용이 최신 버전이라, 영문 내용을 반영해서 새로 업데이트 해주실 수 있으실까요?
그리고 체크박스도 한번씩 체크 부탁드리겠습니다.

@eunwoo-levi
Copy link
Contributor Author

안녕하세요, PR 보내주셔서 감사합니다.

다만, 영문 내용이 최신 버전이라, 영문 내용을 반영해서 새로 업데이트 해주실 수 있으실까요? 그리고 체크박스도 한번씩 체크 부탁드리겠습니다.

@lumirlumir 해당 피드백 반영하여 최신 React 영문 내용 반영한 Commit 및 PR 내용 업데이트 하였습니다.

@eunwoo-levi eunwoo-levi changed the title fix: remove unnecessary english sentences below korean translation fix: remove duplicated English content and improve translation clarity in useActionState docs May 14, 2025
@eunwoo-levi eunwoo-levi requested a review from lumirlumir May 14, 2025 13:03
Copy link
Collaborator

@lumirlumir lumirlumir left a 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) 내에서 수동으로 호출할 수도 있습니다.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
2. `<form>` 컴포넌트의 `action` Prop이나, 폼 내부 `<button>` 컴포넌트의 `formAction` Prop에 전달할 수 있는 새 액션입니다. 이 액션은 [startTransition](/reference/react/startTransition) 내에서 수동으로 호출할 수도 있습니다.
2. `form` 컴포넌트의 `action` Prop이나, 폼 내부 `button` 컴포넌트의 `formAction` Prop에 전달할 수 있는 새 액션입니다. 이 액션은 [`startTransition`](/reference/react/startTransition) 내에서 수동으로 호출할 수도 있습니다.

Copy link
Contributor Author

@eunwoo-levi eunwoo-levi May 14, 2025

Choose a reason for hiding this comment

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

@lumirlumir

해당 리뷰 감사합니다.

리뷰 반영한 최신 commit 확인 부탁드립니다!

@lumirlumir lumirlumir changed the title fix: remove duplicated English content and improve translation clarity in useActionState docs docs: remove duplicated English content and improve translation clarity in useActionState docs May 14, 2025
Copy link
Collaborator

@lumirlumir lumirlumir left a comment

Choose a reason for hiding this comment

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

감사합니다, LGTM!

@lumirlumir lumirlumir merged commit 6c4842a into reactjs:main May 14, 2025
6 checks passed
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.

2 participants