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

[USER] Feat: type store #22

Merged
merged 21 commits into from
Aug 4, 2024
Merged

[USER] Feat: type store #22

merged 21 commits into from
Aug 4, 2024

Conversation

swgvenghy
Copy link
Member

수시, 정시, 편입학 관리하는 zustand 파일 생성하였습니다.
도입한 이유는 다음과 같습니다.

  1. 기본 전제가 수시, 정시, 편입학 셋 중 하나를 선택해야 질문이나 카테고리를 선택할 수 있음.
  2. 질문시, 헤더 상태 등 다양한 곳에서 해당 type을 사용하기에 state관리보다 전역으로 관리하는게 용이하다고 판단하여 zustand도입하여 만들었습니다.

그 외 자잘한 코드 수정 및 page에서 상태 관리 어떤식으로 되는지도 올렸습니다.

@swgvenghy swgvenghy self-assigned this Aug 2, 2024
@swgvenghy swgvenghy requested a review from sangmaaaaan August 2, 2024 06:43
Copy link

github-actions bot commented Aug 2, 2024

Copy link

github-actions bot commented Aug 2, 2024

Copy link

github-actions bot commented Aug 2, 2024

Copy link

github-actions bot commented Aug 2, 2024

Copy link

github-actions bot commented Aug 2, 2024

Copy link

github-actions bot commented Aug 2, 2024

Copy link

github-actions bot commented Aug 2, 2024

Copy link

github-actions bot commented Aug 2, 2024

Copy link
Contributor

@sangmaaaaan sangmaaaaan left a comment

Choose a reason for hiding this comment

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

댓글 남긴 부분 확인부탁드립니다

Copy link
Contributor

Choose a reason for hiding this comment

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

이 부분 설명한번 가능할까요,,?

Copy link
Member Author

Choose a reason for hiding this comment

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

해당 부분 다음 pr올릴때 자세한 설명 덧붙여 말씀드리겠습니다.

@swgvenghy swgvenghy merged commit b71b06f into main Aug 4, 2024
1 check failed
@swgvenghy swgvenghy deleted the USER-type-store branch August 4, 2024 07:47
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.

2 participants