◐ Shell
reader mode source ↗
Skip to content

refactor: vaniilaJS -> React#23

Merged
yongchanson merged 1 commit into
react-migrationfrom
caherin-react
Jun 11, 2022
Merged

refactor: vaniilaJS -> React#23
yongchanson merged 1 commit into
react-migrationfrom
caherin-react

Conversation

@chaerin-dev

Copy link
Copy Markdown
Member
  • 상태관리 없이 구현하는 것이 더 어려워서 Recoil을 이용한 전역상태관리까지 같이 구현하게 되었습니다.
  • React 공부를 따로 할 시간이 없어 지난 미팅 때 @codeisneverodd 님께서 설명해주신 내용과 구글링만을 바탕으로 구현하다보니 부족한 점이 많습니다. 리뷰 부탁드립니다.
  • CSS의 경우 Styled Component로 다시 구현하신다고 하셔서 아예 import 하지 않았습니다.
  • @yongchanson CSS 적용하실 때 로딩컴포넌트 position을 absolute로 해서 나머지 컴포넌트들을 덮도록 해주시면 될 것 같습니다.

@chaerin-dev chaerin-dev added the 프로젝트 TODO Projects 의 해야할 항목을 완료했을 때 사용합니다. label Jun 10, 2022
@chaerin-dev chaerin-dev self-assigned this Jun 10, 2022
@yongchanson yongchanson merged commit 183be94 into react-migration Jun 11, 2022
@yongchanson yongchanson deleted the caherin-react branch June 11, 2022 08:50

@codeisneverodd codeisneverodd left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hide comment

좋은 코드 잘 보았습니다! 고생 많이셨을 것 같네요 👍
리뷰 남겼으니 답글을 남기셔도 좋고, 반영을하셔도 좋을 것 같습니다!!

@chaerin-dev

Copy link
Copy Markdown
Member Author

@codeisneverodd 👍👍👍 리뷰 남겨주신 부분 반영해보도록 하겠습니다!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

프로젝트 TODO Projects 의 해야할 항목을 완료했을 때 사용합니다.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants