Skip to content

docs: fix typo in lifecycle-of-reactive-effects#1511

Merged
lumirlumir merged 1 commit into
reactjs:mainfrom
DongEun02:fix-typo-challenge-description
May 14, 2026
Merged

docs: fix typo in lifecycle-of-reactive-effects#1511
lumirlumir merged 1 commit into
reactjs:mainfrom
DongEun02:fix-typo-challenge-description

Conversation

@DongEun02
Copy link
Copy Markdown
Contributor

index 3572c280..a54ce29b 100644
--- a/src/content/learn/lifecycle-of-reactive-effects.md
+++ b/src/content/learn/lifecycle-of-reactive-effects.md
@@ -1320,7 +1320,7 @@ effect 본문에 `console.log('Resubscribing')`를 추가하면 이제 체크 
 
 이 예시에서 `chat.js`의 채팅 서비스는 `createEncryptedConnection`과 `createUnencryptedConnection`이라는 두 개의 서
로 다른 API를 노출합니다. 루트 `App` 컴포넌트는 사용자가 암호화 사용 여부를 선택할 수 있도록 한 다음, 해당 API 메서
드를 하위 `ChatRoom` 컴포넌트에 `createConnection` prop으로 전달합니다.
 
-처음에는 콘솔 로그에 연결이 암호화되지 않았다고 표시됩니다. 체크 박스를 켜면 아무 일도 일어나지 않습니다. 그러나 그 후에 선택한 대화방을 변경하면 채팅이 다시 연결*되고* 콘솔 메시지에서 볼 수 있듯이 암호화가 활성화됩니다. 이것은 버
그입니다. 체크 박스를 토글해*도* 채팅이 다시 연결되도록 버그를 수정했습니다.
+처음에는 콘솔 로그에 연결이 암호화되지 않았다고 표시됩니다. 체크 박스를 켜면 아무 일도 일어나지 않습니다. 그러나 그 후에 선택한 대화방을 변경하면 채팅이 다시 연결*되고* 콘솔 메시지에서 볼 수 있듯이 암호화가 활성화됩니다. 이것은 버
그입니다. 체크 박스를 토글해*도* 채팅이 다시 연결되도록 버그를 수정하세요.

변경사항: "수정했습니다" -> "수정하세요"

원문이 독자에게 작업을 수행하도록 요구하는 명령문입니다 ! (챌린지 문제)
완료형인 "수정했습니다"보다 지시형인 "수정하세요"가 원문의 의도와 문맥에 더 정확할거 같아요 :)

원문: https://react.dev/learn/lifecycle-of-reactive-effects#challenges (챌린지 문제4번 )

필수 확인 사항

선택 확인 사항

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

Co-authored-by: Antoliny0919 <antoliny0919@gmail.com>
Co-authored-by: TH-97 <hood3392@gmail.com>
Co-authored-by: GGGGGGGGGGGGGGG_J <GamjaIsMine02@users.noreply.github.com>
Co-authored-by: lee-eojin <lejin0906@gmail.com>
@vercel
Copy link
Copy Markdown

vercel Bot commented May 14, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
ko-legacy-reactjs-org Ignored Ignored Preview May 14, 2026 9:07am

Request Review

Copy link
Copy Markdown
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, 감사합니다!

@github-actions
Copy link
Copy Markdown

Size changes

Details

📦 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 c9f3585 into reactjs:main May 14, 2026
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