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.

Sign up for GitHub

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

Jump to bottom

Fix: 프록시 타겟 주소 변경 #82

Merged
merged 1 commit into from
Nov 28, 2023
Merged

Fix: 프록시 타겟 주소 변경 #82

merged 1 commit into from
Nov 28, 2023

Conversation

2YH02
Copy link
Contributor

@2YH02 2YH02 commented Nov 28, 2023

Issue Number

ETC

  • 타켓 주소만 변경하였습니다. 환경 변수로 적용하려 했는데 잘 안되서 일단 다같이 공통적으로 작업하기 위해 pr 올립니다!
  • axios요청을 그냥 보낼 때는 baseURL까지 작성 해주셔야 되고 useAxios를 사용하면 baseURL 제외 하고 작성 가능합니다.
    따라서 axios 요청 있는 부분을 useAxios에서 인스턴스 불러와서 하는것을 권장 드립니다.
  • 환경변수가 변경 되었습니다 VITE_APP_API_BASE_URL=/api 이런식으로 고쳐주세요!!

suyeonnnnnnn
suyeonnnnnnn approved these changes Nov 28, 2023
View reviewed changes
Copy link
Collaborator

@suyeonnnnnnn suyeonnnnnnn left a comment

Choose a reason for hiding this comment

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

넵! 감사합니다~!

LikeFireAndSky
LikeFireAndSky approved these changes Nov 28, 2023
View reviewed changes
Copy link
Collaborator

@LikeFireAndSky LikeFireAndSky left a comment

Choose a reason for hiding this comment

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

감사합니다!! 바로 적용해보도록 하겠습니다

@LikeFireAndSky LikeFireAndSky merged commit 9c32016 into develop Nov 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@LikeFireAndSky LikeFireAndSky LikeFireAndSky approved these changes

@suyeonnnnnnn suyeonnnnnnn suyeonnnnnnn approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@2YH02 @suyeonnnnnnn @LikeFireAndSky