Skip to content

[리팩토링] 상태관리 관련 리팩토링 #29

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.

Sign up for GitHub

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

Jump to bottom
Merged
chaerin-dev merged 8 commits into from
Jun 28, 2022
Merged

[리팩토링] 상태관리 관련 리팩토링 #29

chaerin-dev merged 8 commits into from
Jun 28, 2022
+42 −42

Conversation

@chaerin-dev
Copy link
Member

@chaerin-dev chaerin-dev commented Jun 27, 2022 •
edited
Loading

개요

  • 상태관리 관련 리팩토링

작업 내용

  • refactor: 필요 없는 import 제거
  • refactor: 상태 선언 방식 let -> const 변경
  • refactor: setter 함수 미 이용시 useRecoilState -> useRecoilValue 변경
  • refactor: useRecoilValue와 useSetRecoilState -> useRecoilState 변경

관련 이슈

참고 자료

@chaerin-dev chaerin-dev added the 프로젝트 TODO Projects 의 해야할 항목을 완료했을 때 사용합니다. label Jun 27, 2022
@chaerin-dev chaerin-dev self-assigned this Jun 27, 2022
@codeisneverodd
Copy link
Member

codeisneverodd commented Jun 28, 2022

코드 잘 보았습니다! 메이저한 로직 변동은 없어서 그대로 머지 하셔도 좋을 것 같습니다! squash merge 이용해주세요!!

@chaerin-dev chaerin-dev changed the title [리팩토링] styled-componets 이름 및 구조 변경 [리팩토링] 상태관리 관련 리팩토링 Jun 28, 2022
@chaerin-dev chaerin-dev merged commit 3dea304 into develop Jun 28, 2022
@chaerin-dev chaerin-dev deleted the refactor-refactoring-chaerin-dev branch June 28, 2022 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@chaerin-dev chaerin-dev

Labels

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

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@chaerin-dev @codeisneverodd