Merged
Conversation
bk-git-hub
reviewed
Mar 26, 2026
Collaborator
bk-git-hub
left a comment
bk-git-hub
left a comment
There was a problem hiding this comment.
다혜님 정말 고생 많았습니다!
지난주 피드백이 전부 잘 반영돼있네요 :)
이번주 핵심이었던 props로 완료 상태를 전달하는 부분도 잘 구현해주셨습니다
커밋도 작업단위로 해보려고 고민하신 흔적이 보입니다
다음주에는 커밋에 prefix와 함께 구체적인 작업 내용을 적어주시면 더 보기 좋은 커밋 메세지가 될 수 있을 것 같아요!
Comment on lines
+18
to
+26
| <div className="container"> | ||
| <TodoList todos={todos} /> | ||
| </div> | ||
|
|
||
| <div className="frame2"> | ||
| <TodoHeader /> | ||
| </div> | ||
| <div className="container"> | ||
| <TodoList /> | ||
| <TodoList todos={[]} /> |
Collaborator
There was a problem hiding this comment.
다음주에는 리스트가 비었을경우를 조건으로 조건부 렌더링을 해보면 좋을 것 같아요
Comment on lines
+6
to
+12
| const todos = [ | ||
| { id: 1, text: "리액트 공식문서 읽기", isDone: true }, | ||
| { id: 2, text: "알고리즘 문제 풀기", isDone: true }, | ||
| { id: 3, text: "운동 30분 하기", isDone: false }, | ||
| { id: 4, text: "프로젝트 회의 준비", isDone: false }, | ||
| ]; | ||
|
|
Collaborator
There was a problem hiding this comment.
todos 상수는 별도의 파일로 관리를 한다면 App.tsx 의 가독성이 더 좋아질 것 같아요!
파일 이름의 경우는 취향에 따라 다양한데 몇가지 예시를 들자면
todo.data.ts
todos.data.ts
todo.mock.ts
mockTodos.ts
이정도가 있을거같네요
Comment on lines
+3
to
+7
| interface Todo { | ||
| id: number; | ||
| text: string; | ||
| isDone: boolean; | ||
| } |
Collaborator
There was a problem hiding this comment.
Todo interface는 현재 TodoList 뿐만 아니라 다른 곳에서도 활용될 여지가 있어보여요. 실제로 공용타입을 src/types 에
todo.types.ts 이런식으로 파일을 따로 만들어 interface나 type만 모아서 관리하는 경우도 있답니다
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
✅ 제출 정보
--- 체크박
✅ 체크리스트 (필수)
main이 아니라 내 GitHub 핸들 브랜치(<handle>)다<handle>-week-xx형식이다🧩 구현 내용 요약
❓ 궁금한 점