search

ddoyongida / my-todolist-level-2

0 stars 0 forks source link

리액트 LV2 피드백 - 예병수 튜터 #1

Open DevelopSoo opened 1 year ago

DevelopSoo commented 1 year ago

프로젝트 자체에서 에러가 발생하여 제대로 피드백을 할 수 없었습니다. 아마 바쁘셔서 수정을 못하신 것 같습니다. 다음 프로젝트엔 정확하게 에러를 해결하고 올려주시면 좋을 것 같습니다.

화면은 제외하고 코드만 보고 피드백을 드리겠습니다.

DevelopSoo commented 1 year ago

https://github.com/ddoyongida/my-todolist-level-2/blob/f1fdf96b21ad6635a98e2fa3d7e3076110d79b57/src/redux/modules/reducers/todoReducers.js#L21-L31

redux 구현시 case 우측의 type은 변수로 사용하는 것이 좋습니다. 그 이유는 action creator 부분에서도 중복되기 때문입니다. 변수의 사용 이유가 무엇인지 확인해보시면 좋을 것 같습니다.

DevelopSoo commented 1 year ago

https://github.com/ddoyongida/my-todolist-level-2/blob/f1fdf96b21ad6635a98e2fa3d7e3076110d79b57/src/redux/modules/reducers/todoReducers.js#L1-L17

todo의 isDone 속성을 바탕으로 데이터를 변경하는 것이 원래 목적이었습니다. doneTodo를 만들 필요가 없을 것 같습니다.

DevelopSoo commented 1 year ago

상세페이지를 구현하지 못하신 것 같습니다. 다음 프로젝트에서 많이 어려우신 경우 튜터님들의 도움을 조금 받는 것이 좋을 것 같습니다.