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

[NDD-86] 상태관리 라이브러리 추가 (순수 세팅 시간 1h/8h) (상태 관리 고민시간 4h/???) #25

Merged
merged 5 commits into from
Nov 10, 2023

Conversation

Yoon-Hae-Min
Copy link
Collaborator

@Yoon-Hae-Min Yoon-Hae-Min commented Nov 9, 2023

NDD-86 Powered by Pull Request Badge

Why

전역 상태를 관리하기 위한 라이브러리를 추가했습니다. 설정페이지에 상태를 관리하는것은 여러 컴포넌트 및 페이지를 넘나드는 상태가 필요했기 때문에 전역 상태라이브러리를 채택하였습니다.

How

axios 기존에 쓰던 기본 설정을 때왔습니다. 간단하게 디버깅하기 좋을 정도로만 만들어 놨습니다.

Reference

상태관리 고민하면서 사용했던 레퍼런스 입니다.
https://velog.io/@kandy1002/%EC%88%98%EC%A0%9C-useFunnel%EB%A1%9C-%ED%8D%BC%EB%84%90-%EB%8D%B0%EC%9D%B4%ED%84%B0-%EC%9D%80%EB%8B%89%ED%99%94%ED%95%98%EA%B8%B0

ETC

진행하면서 상태관리에 대해서 생각해 봤는데 이건 한번 만나서 pre commit Review를 해보면 좋을것 같습니다. 세팅 페이지 경로 관리와 문제 컴포넌트의 상태관리입니다.
이게 구조만 잡다보니까 에러가 엄청나서 PR에 코드를 적용하지는 못하겠더라고요 그래서 따로 구조만 적어두고 세팅한것만 올립니다.

@Yoon-Hae-Min Yoon-Hae-Min added the FE 프론트엔드 코드 변경사항 label Nov 9, 2023
@Yoon-Hae-Min Yoon-Hae-Min self-assigned this Nov 9, 2023
@Yoon-Hae-Min Yoon-Hae-Min changed the title [NDD-22] 상태관리 라이브러리 추가 (순수 세팅 시간 1h/8h) (상태 관리 고민시간 4h/???) [NDD-86] 상태관리 라이브러리 추가 (순수 세팅 시간 1h/8h) (상태 관리 고민시간 4h/???) Nov 9, 2023
<Global styles={_global} />
<AppRouter />
</ThemeProvider>
<QueryClientProvider client={queryClient}>
Copy link
Collaborator

Choose a reason for hiding this comment

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

이건 그냥 궁금한건데 react-query와 recoil의 wrapping 순서가 의미가 있을까요?!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

별 의미는 없습니다~

Copy link
Collaborator

@milk717 milk717 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

@quiet-honey
Copy link
Collaborator

Axios를 봐서 반가웠습니다ㅎㅎㅎ
LGTM:)

@Yoon-Hae-Min Yoon-Hae-Min merged commit f4aa63a into dev Nov 10, 2023
@delete-merged-branch delete-merged-branch bot deleted the feature/NDD-86 branch November 10, 2023 13:42
Yoon-Hae-Min added a commit that referenced this pull request Nov 11, 2023
* chore: axios 라이브러리 추가

* chore: axios 기본 instance 생성

* chore: tanstack query 라이브러리 추가

* chore: recoil 라이브러리 추가

* chore: tanstack query devtools 추가
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FE 프론트엔드 코드 변경사항
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants