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

[혜수] - 인스턴스 생성 #58

Merged
merged 4 commits into from
Nov 15, 2023
Merged

[혜수] - 인스턴스 생성 #58

merged 4 commits into from
Nov 15, 2023

Conversation

suehdn
Copy link
Contributor

@suehdn suehdn commented Nov 14, 2023

📌 이슈 번호

close #55

🚀 구현 내용

  • localstorage -> cookie로 변경
  • axiosInstance 생성
  • useRefreshNameMutation, useAllPlansQuery 임시 생성

📘 참고 사항

노션에 컴포넌트 사용 설명 써놨습니다

노철님
쿠키로 변경하면서 useAuth 부분 state가 이제 필요 없을 수도 있을 것 같습니다. 한번 확인해보시면 좋을 것 같아요.

@suehdn suehdn self-assigned this Nov 14, 2023
@suehdn suehdn added the 🚨긴급 긴급한 확인 필요 label Nov 14, 2023
Copy link
Contributor

@qkdl60 qkdl60 left a comment

Choose a reason for hiding this comment

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

결국 해내셨네요👍
useAuth도 수정하도록 하겠습니다

Copy link
Contributor

Choose a reason for hiding this comment

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

👍 store에도 적용해주셨군요

Copy link
Contributor

@MinwooP MinwooP left a comment

Choose a reason for hiding this comment

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

혜수님 너무 고생하셨습니다 !! 완벽히 이해하진 못했지만 우선 빨리 머지하고 사용하셔야 하니 차근차근 읽어보고 궁금한 점 생기면 질문할게여 !!

@suehdn suehdn merged commit 6c3cdf4 into dev Nov 15, 2023
@suehdn suehdn deleted the feat/#55/instance branch November 15, 2023 01:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🚨긴급 긴급한 확인 필요
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[혜수] - 인스턴스 생성
3 participants