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

Feign통신연결 및 프론트 요구사항 반영 #26

Merged
merged 25 commits into from
Dec 9, 2023
Merged

Conversation

qwerty1434
Copy link
Contributor

개요

  1. Feign통신 설정
  2. 프론트 요구사항 반영
  3. 테스트컨테이너 적용

PR 유형

어떤 변경 사항이 있나요? ✅

content
새로운 기능 추가
버그 수정
CSS 등 사용자 UI 디자인 변경
코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
코드 리팩토링
주석 추가 및 수정
문서 수정
테스트 추가, 테스트 리팩토링
빌드 부분 혹은 패키지 매니저 수정
파일 혹은 폴더명 수정
파일 혹은 폴더 삭제

스크린 샷

PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • local에서 실행되는 것을 확인 했나요?
  • 변경 사항에 대해 테스트를 통과했나요?
  • 설계와 관련된 변경 사항을 공유했나요? (api 스펙, entity 등)

매번 요청하는 게 비효율적이고 변경이 발생해도 괜찮은 데이터라 반정규화 진행
일반적인 쇼핑몰에 잘 존재하지 않는 개념으로 서비스 전체에 불필요하다는 팀 전체의 판단으로 삭제
@nowgnas nowgnas merged commit 6a94220 into develop Dec 9, 2023
1 check passed
@qwerty1434 qwerty1434 deleted the LF1-554-Feign branch December 9, 2023 02:36
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.

2 participants