Skip to content

Conversation

@Glyph8
Copy link
Contributor

@Glyph8 Glyph8 commented May 1, 2025

📚 워크북 링크

https://fantasy-paper-18d.notion.site/5-Thinking-in-React-1e369d4659b68098bfe1ed3c0fb94c64?pvs=4

📝 변경/추가 사항 요약

  • 항목 삭제 기능 구현
  • 항목 수정 기능 구현

✅ 리뷰 요구사항

리팩토링은 어떤 방식을 기대한 문제였는지 궁금합니다.

📸 확인 방법 (스크린샷)

image [추가] image [수정] image [삭제]

<제출 전 체크리스트>

  • week00 폴더 내에 자신의 폴더를 새로 만드셨나요?
  • 자신 폴더 외의 다른 사람의 폴더에서 코드를 수정하지 않았나요?

Copy link
Member

@junghyungo junghyungo left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

Copy link
Member

@Turtle-Hwan Turtle-Hwan left a comment

Choose a reason for hiding this comment

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

리팩토링은 현재 분리한 컴포넌트가 최선인지,
상태를 너무 깊게 전달하는 것은 아닌지,
추가로 더 분리할 컴포넌트가 있는지 or 너무 잘게 분리한 것은 아닌지 고민해보면 좋을 것 같다는 의도였습니다!

@Turtle-Hwan Turtle-Hwan merged commit 7f1b0fc into main May 8, 2025
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants