Skip to content

Conversation

@mlnwns
Copy link
Contributor

@mlnwns mlnwns commented Feb 2, 2026

extracting-state-logic-into-a-reducer.md의 typo 수정

case 'sent_message': {
      return {
        ...state,
        message: ''
      };
    }

messengerReducer 예시 코드에서는 sent_message라는 케이스명을 사용하고 있습니다.

설명하는 문서에서는 이를 반영하기 위해 send_message라는 새로운 action을 만들어서 reducer에서 별도로 분리하여 작성해보겠습니다. 처럼 send_message라는 케이스명을 사용하고 있어 일관성을 위해 sent_message로 수정하였습니다.

필수 확인 사항

선택 확인 사항

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

Copilot AI review requested due to automatic review settings February 2, 2026 10:06
@vercel
Copy link

vercel bot commented Feb 2, 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 Feb 2, 2026 10:06am

Request Review

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Fixes a typo/inconsistency in the “Extracting State Logic into a Reducer” doc by aligning the action type name in the explanation with the example reducer code (sent_message).

Changes:

  • Updated the narrative text to reference sent_message (instead of send_message) to match the accompanying example code.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-actions
Copy link

github-actions bot commented Feb 2, 2026

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

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 403bcc0 into reactjs:main Feb 2, 2026
11 of 12 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