Skip to content

Conversation

@hardwoong
Copy link
Member

📝 미션 번호

2주차 Misson 1, 2

📋 구현 사항

  • 이전에 구현한 TodoList를 React를 사용하여 리팩토링
  • 간단한 페이지를 useContext를 사용하여 라이트 모드, 다크 모드 구현

📎 스크린샷

2025-03-24.7.13.27.mov
2025-03-24.7.12.41.mov

✅ 체크리스트

  • Merge 하려는 브랜치가 올바르게 설정되어 있나요?
  • 로컬에서 실행했을 때 에러가 발생하지 않나요?
  • 불필요한 주석이 제거되었나요?
  • 코드 스타일이 일관적인가요?

🤔 질문 사항

@hardwoong hardwoong self-assigned this Mar 24, 2025
Copy link
Member

@hyesngy hyesngy left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 👍🏻👍🏻

컴포넌트별로 독립된 폴더를 생성하고 그 안에 index.tsx 파일을 배치하는 구조적 접근 방식이 인상적입니다. 이런 방식은 import 경로를 간결하게 만들고, 큰 규모의 프로젝트에서 파일을 관리하기 좋습니다!

이번 주차에서 학습한 useState, Context API를 앞으로도 잘 활용할 수 있었으면 좋겠습니다!

Copy link
Member

Choose a reason for hiding this comment

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

TodoListDoneList 컴포넌트가 구조적으로 매우 유사한데, 두 개의 파일로 분리하기보다 이 둘을 하나의 재사용 가능한 컴포넌트로 합치면 코드 중복을 줄일 수 있고 유지보수성을 높일 수 있을 것 같아요!

@hardwoong hardwoong merged commit 15f6180 into main Mar 31, 2025
1 check 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.

3 participants