Skip to content
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: remove ref guide about react18 #1140

Merged
merged 2 commits into from
Feb 18, 2025

Conversation

nayounsang
Copy link
Contributor

@nayounsang nayounsang commented Feb 18, 2025

useRef에서 forwardRef에 대한 설명을 지우고 콘솔 에러메시지를 최신 React19 문서에 맞게 반영

  • forwardRef에 대한 설명은 React19이후엔 필요없어지고 영문 문서에도 해당 내용이 빠져 뺐습니다.
  • 단순 ref를 props로 넘겨준다는 미번역된 문구 번역했습니다.
  • 콘솔 에러메시지가 나타나지 않았고 영문 문서와 동일한 콘솔 에러메시지를 표시하도록했습니다. (기존엔 에러메시지가 보이지 않았네요.)

필수 확인 사항

선택 확인 사항

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

Copy link

vercel bot commented Feb 18, 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 Feb 18, 2025 11:13am

@nayounsang nayounsang changed the title docs: renove ref guide about react18 docs: remove ref guide about react18 Feb 18, 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

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! 🙌

@lumirlumir lumirlumir merged commit 234d8df into reactjs:main Feb 18, 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