search

songjuu / todolist-leve1-2

- 개인 과제: TodoList 만들기 Lv.1+Lv.2
https://todolist-leve1-2.vercel.app
0 stars 0 forks source link

Feedback #1

Open wildCodingWarrior opened 1 year ago

wildCodingWarrior commented 1 year ago

Overview

wildCodingWarrior commented 1 year ago

https://github.com/songjuu/todolist-leve1-2/blob/4a01c64baef58dc92163204448c038640d2f9526/src/redux/modules/todos.js#L7-L33

ducks 패턴에서 action이란 type과 payload로 이루어진 객체입니다. 추후 다른 개발자와 협업할 때는 해당 디자인 패턴을 따르는 편이 좋겠습니다.

wildCodingWarrior commented 1 year ago

https://github.com/songjuu/todolist-leve1-2/blob/4a01c64baef58dc92163204448c038640d2f9526/src/components/InputForm.jsx#L13-L24

유효성 검사를 추가하면 좋겠습니다.

wildCodingWarrior commented 1 year ago

https://github.com/songjuu/todolist-leve1-2/blob/4a01c64baef58dc92163204448c038640d2f9526/src/pages/TodoDetail.jsx#L33-L57

지금도 좋지만, 선발대에서 스타일을 분리하는 컨벤션을 소개한 적 있습니다. 참고하면 좋을 것 같습니다. 링크