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 unnecessary state reset in List component #1135

Merged
merged 1 commit into from
Feb 4, 2025

Conversation

ByungyeonKim
Copy link
Contributor

배열 State 업데이트하기 페이지 내 코드 수정

안녕하세요! 배열 State 업데이트하기 페이지의 배열에 항목 추가하기 부분에서 클릭 이벤트 내 setName(''); 코드를 제거했습니다.

새로운 배열로 추가했을 때와 동일하게 작동해서 확인해보니, set 함수를 통해 리렌더링을 시켜주고 있었습니다. 원문과 동일하게 setName(''); 코드 부분을 제거하였습니다!

필수 확인 사항

선택 확인 사항

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

Copy link

vercel bot commented Feb 4, 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 4, 2025 6:05am

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 changed the title fix: remove unnecessary state reset in List component docs: remove unnecessary state reset in List component Feb 4, 2025
Copy link

github-actions bot commented Feb 4, 2025

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 9baceed into reactjs:main Feb 4, 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