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

[Feat/#36] api 환경 instance 기본 세팅 #38

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

GraceKim527
Copy link
Contributor

🔥 Related Issues

⛅️ 작업 내용

  • api 인스턴스 기본 세팅 해두었습니다.

✅ PR Point

  • apis 폴더를 생성하여 다음과 같이 관리하면 어떨까합니다!
    • apis/utils : 기본 axios 인스턴스 세팅
    • apis/api : api 요청 및 응답에 필요한 요소
    • apis/services : 응답결과가 정제가 필요한 경우 사용
  • env파일에 추후 REACT_APP_BASE_URL 요소 추가해야할 것 같습니다.
  • 토큰 받는 부분이나 요청/응답에 필요한 인터셉터도 함께 세팅해두었는데, 필요 시 instance파일에서 없애고 다른 파일에서 사용해도 될 것 같습니다.

@GraceKim527 GraceKim527 requested a review from eunbeann September 5, 2024 08:06
@GraceKim527 GraceKim527 self-assigned this Sep 5, 2024
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.

[Feat] API 인스턴스 환경 생성
1 participant