[Infra] dev 브랜치 main으로 병합 #310
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
fix: server url을 infra 설정의 helm chart에 매핑된 value로 수정
|
dev에서 작업한 내용을 배포하려면 main으로 병합을 해주어야 해요! 이전에 공유했던 cors 문제 해결을 위한 환경 변수 네이밍 컨벤션을 맞추기 위해서 URL_SERVER -> SERVER_SERVICE_URL 로 변경하기도 했고 로컬에서 배포 서버 API 사용을 위해서 cors 허용 설정을 추가하려는 작업을 더하고 다시 병합을 하면 좋을 것 같아요. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
📝작업 내용
배포를 위해 dev 브랜치를 main 브랜치로 병합합니다