Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[NEXTLEVEL 클린코드 리액트 김윤건] 장바구니 미션 Step3 #13

Open
wants to merge 2 commits into
base: younchong
Choose a base branch
from

Conversation

younchong
Copy link

구현사항

  • step2하면서 미리 구현했던 사항들이라서 새로 고친부분은 많이 없습니다. (상품상세등)
  • Redux를 다시 공부하고 조금 이해하게 돼서 Redux사용한 부분은 통일되지 않았던 부분을 정리하고 틀린부분들은 고쳤습니다.
  • Redux-saga도 추가해보고 싶었지만, 어떤 부분에서 필요로 한지 판단이 안서서 추가하지 못하고, 마무리했습니다.

정리

  • 다시 공식문서를 보면서 공부를 하니까 예전에 구현을 위해서만 짰던 코드들이 잘못됐음을 많이 깨닫고 부족함도 많이 깨닫게 된 것 같습니다. 그래도 늦었지만 마무리 짓고 싶어서 redux부분만 조금 정리해서 올립니다.

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.

1 participant