Skip to content

docs: Translate remaining English sentences in useId.md and useLayoutEffect.md to Korean #1238

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 2 commits into from
Jul 12, 2025

Conversation

ohjooyeong
Copy link
Contributor

번역: useId.md와 useLayoutEffect.md의 남은 영어 문장들 한국어로 번역

변경 사항

두 개의 레퍼런스 문서에서 번역되지 않은 영어 문장들을 한국어로 번역했습니다.

src/content/reference/react/useId.md

  • "useId currently cannot be used in async Server Components"
    → "useId는 현재 비동기 서버 컴포넌트에서 사용할 수 없습니다"
  • "You do not need to pass identifierPrefix if you only have one React app on the page"
    → "페이지에 React 앱이 하나만 있는 경우에는 identifierPrefix를 전달할 필요가 없습니다"

src/content/reference/react/useLayoutEffect.md

  • "If you trigger a state update inside useLayoutEffect, React will execute all remaining Effects immediately including useEffect"
    → "useLayoutEffect 내부에서 state 업데이트를 실행하면 React는 useEffect를 포함한 나머지 모든 Effect를 즉시 실행합니다"

필수 확인 사항

선택 확인 사항

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

Copy link

vercel bot commented Jul 12, 2025

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

Name Status Preview Comments Updated (UTC)
ko-legacy-reactjs-org 🔄 Building (Inspect) Visit Preview 💬 Add feedback Jul 12, 2025 4:36am

Copy link

Size changes

📦 Next.js Bundle Analysis for undefined

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

This PR introduced no changes to the JavaScript bundle! 🙌

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 caf82f1 into reactjs:main Jul 12, 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