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

[hotfix/#102/UserInfo] 유저 정보 저장 로직 수정 #103

Merged
merged 9 commits into from
Mar 12, 2024

Conversation

wdgWon
Copy link
Contributor

@wdgWon wdgWon commented Mar 11, 2024

#️⃣연관된 이슈

close #102

💡 핵심적으로 구현된 사항

  • 토큰이 정상적으로 Authorization에 들어가지 않던 문제를 수정하였습니다.
  • 엔드포인트 변경에 의한 url 변경하였습니다.
  • authToken에서 초기값으로 토큰이 아닌 공백문자를 저장하던 문제를 수정하였습니다.

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

  • 커스텀 에러를 통해 에러 핸들링을 구현하였습니다.

▶️ 테스트,검증

  • 엔드포인트나 응답값 변경으로 수정된 부분들을 제외한 authToken 로직과 axiosInstance 부분을 봐주시면 됩니다!

🤔 고민 사항

  • 그리고 customError로 에러 바운더리에서 해당 에러가 커스텀에러의 인스턴스인지 판단하여 에러 핸들링하려 합니다! 혹시 더 좋은 아이디어나 참고할만한 레퍼런스가 있으면 공유 부탁드립니다~!
  • 중복 코드는 나중에 리팩토링 하겠습니다 ㅎㅎ;;;

vite.config.ts 가 수정되어 이제 기본적으로 서버로 api 요청을 보냅니다!!

  • 혹시 msw로 전환하고 싶으신분은 baseUrl을 공백으로 바꾸시면 됩니다~
  • 더 자세한건 노션 - 프론트엔드 - 논의사항을 참고해주세요!

@wdgWon wdgWon added HotFix 버그 픽스 Refactor 리팩토링 Urgent 긴급 labels Mar 11, 2024
@wdgWon wdgWon self-assigned this Mar 11, 2024
@wdgWon wdgWon merged commit 783a6fc into dev Mar 12, 2024
1 check passed
@wdgWon wdgWon deleted the Hotfix/#102/UserInfo branch March 12, 2024 01:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
HotFix 버그 픽스 Refactor 리팩토링 Urgent 긴급
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] 유저 정보 가져오지 못하는 에러
3 participants