-
Notifications
You must be signed in to change notification settings - Fork 2
Feature/#55 axios instance #56
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
Conversation
…tFE11-part3-team7/wikid into feature/#55_Axios-Instance
haksoo0918
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
api 함수들이 이뿌네요. 👍
env 파일 만들어야 겠군요. 혹시 api url 말고 더 추가한게 있으시면 공유 부탁드립니다.
모달과 api 할수랑은 따로 구분해서 올려 주시는게 맞는것 같습니다.
junghwaYang
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
깔끔하게 잘 작성된 것 같습니다 👍🏻 수고하셨어요!!
@crazyupinc-design 지이님도 확인 부탁드려요~
crazyupinc-design
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
확인했습니다 👍🏻수고많으셨습니다
|
@horororok |
이슈 번호
close #55
변경 사항 요약
클라이언트 컴포넌트에서 사용할 axios 인스턴스입니다.
테스트 결과
베이스(develop) 브랜치에 포함되기 위한 코드는 모두 정상적으로 작동이 되어야 합니다.
ex) 결과물에 대한 스크린 샷
✨ 토큰 관리와 요청/응답 인터셉터가 포함된 커스텀 Axios 인스턴스 추가
이 커밋만 보시면 됩니다!