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

[feature/#3/Store] 전역 상태 관리 초기 설정 #27

Merged
merged 14 commits into from
Feb 17, 2024

Conversation

Whoknow77
Copy link
Contributor

@Whoknow77 Whoknow77 commented Feb 15, 2024

#️⃣연관된 이슈

#3

💡 핵심적으로 구현된 사항

image
  • 전역 스토어 설정(user객체, 로그인, 로그아웃, 유저 업데이트 함수)
  • 새로고침 시 값 유지되도록 설정
  • test페이지에 코드를 작성해놨고 merge전에 지울 예정입니다.
  • 토큰에 대한 상태값을 class로 관리하는 이유는 store에서 hook에 접근할 수 없어서 입니다. 최대한 로그인을 호출하는 쪽은 내부의 로직을 모르도록 하기 위해 그렇게 하였습니다.

https://yescoding.tech/frontend/access-token-global-state

➕ 그 외에 추가적으로 구현된 사항

🤔 테스트,검증 && 고민 사항

📌 PR Comment 작성 시 Prefix for Reviewers

@Whoknow77 Whoknow77 added the Feat 기능 구현 label Feb 15, 2024
@Whoknow77 Whoknow77 added this to the 1차 스프린트 milestone Feb 15, 2024
Copy link
Contributor

@wdgWon wdgWon left a comment

Choose a reason for hiding this comment

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

고생하셨습니다~
한 번 쭉 훑어보고 또 리뷰 달겠습니다!

@Whoknow77 Whoknow77 merged commit aac479e into dev Feb 17, 2024
@Whoknow77 Whoknow77 deleted the feature/#3/Store branch February 17, 2024 13:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feat 기능 구현
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants