-
Notifications
You must be signed in to change notification settings - Fork 1
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
Develop #65
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
객체를 내부에서 찾지 않고 서비스 레이어에서 가져온 후 매개변수로 전달
가게 도메인 수정 #LF1-417
나중에 일정 여유가 있다면 Persistable을 이용한 코드로 변경해 볼 예정
프론트로부터 받은 level로 거리를 결졍하며, meter단위로 변경
도메인 공통 수정 #LF1-436
주소와 배송정책은 가게와 함께 생성되어야 하므로 Service계층에서 각각 만들지 않고 storeCreator에서 함께 생성
프론트 연결 중 수정사항
✏️ 재고 부족으로 인한 주문 실패 시 발송하는 문자 메시지 변경
✏️ 에러 메시지 수정
✨ 정산을 위한 가게정보 반환 feign api 작성
✏️ 가게 정산정보를 주고받는 Dto 변경
✏️ QA 수정사항 반영
✏️ 쿠폰은 현재 사용 가능한 쿠폰만 유저에게 노출된다
✏️ 쿠폰 동시성 테스트를 위해 로그인 토큰이 필요없는 쿠폰 발급 api 생성
✏️ 쿠폰 QA 후 문제점 수정
✏️ 가게사장의 쿠폰확인 시 발급수량도 함께 전달
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.
개요
PR 유형
어떤 변경 사항이 있나요? ✅
스크린 샷
PR Checklist
PR이 다음 요구 사항을 충족하는지 확인하세요.