This repository has been archived by the owner on Aug 13, 2022. It is now read-only.
#93 카테고리 mybatis -> jpa #108
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
f-lab-bright
reviewed
Aug 2, 2021
Comment on lines
13
to
14
| @Entity | ||
| public class ShopCategory { |
There was a problem hiding this comment.
shop - category의 매핑 테이블을 Entity로 만들어서 사용하는게 최선일까요..?
객체 입장에서는 shop이 어떤 카테고리에 속하는지만 알 수 있으면 될텐데, ShopCategory라는 Entity를 이용하는 방법 밖에는 없을까요?
|
Shop이 Category와 관계는 있지만, 직접 참조할 일은 없다고 생각되어서 Category의 id만 가지도록 수정했습니다!! |
lyh7712
approved these changes
Aug 7, 2021
soongjamm
added a commit
that referenced
this pull request
Aug 7, 2021
* root package bluedelivery로 변경 * common.authentication 패키지 정리 * user package * api-application-domain-infra * config -> common.config * PasswordValidator를 controller와 같은 패키지로 이동 * test package * (AddressService) application -> domain 으로 이동 * category package * shop package * businesshour package * closingday package * menu 패키지 정리 * api(menu) 패키지 분할 * category adapter package * #101 @AuthenticationRequired + ArgumentResolver 이면 redis를 2번 조회하는 문제 (#102) * fix: 한 요청내에서 @AuthenticationRequired로 인증객체를 한번 조회하면 ThreadLocal에 저장하여 재사용 * ThreadLocal을 검사하도록 수정 * refactor: business hour * refactor: BusinessHour를 값 타입으로 변경 * (shop): mybatis mapper 제거 * #93 카테고리 mybatis -> jpa (#108) * refactor: ShopCategoy 엔티티 추가 및 Category를 JPA로 * fix controller PathVariable error * ShopCategoryRepository moved into domain package * fix query typo * jmeter script * #96 jmeter script (#109) * jmeter script * util: jib 설정 추가, redis 설정 수정, docker-compose 수정 등 * util: jib 설정 추가, redis 설정 수정, docker-compose 수정 등 (#110) * import 한줄 제거 Co-authored-by: lyh7712 <mmj2rv9@kakao.com>
soongjamm
added a commit
that referenced
this pull request
Aug 13, 2021
* feat: 기본적인 주문 예외 발생 상황 * test: 예외 케이스 추가 * refactor: OrderTest * fix: MenuRepository JPA 상속 제거(아직 Menu Entity 사용 불가능) * #76 #93 영업시간 구현 수정, jpa 적용 (#106) * root package bluedelivery로 변경 * common.authentication 패키지 정리 * user package * api-application-domain-infra * config -> common.config * PasswordValidator를 controller와 같은 패키지로 이동 * test package * (AddressService) application -> domain 으로 이동 * category package * shop package * businesshour package * closingday package * menu 패키지 정리 * api(menu) 패키지 분할 * category adapter package * #101 @AuthenticationRequired + ArgumentResolver 이면 redis를 2번 조회하는 문제 (#102) * fix: 한 요청내에서 @AuthenticationRequired로 인증객체를 한번 조회하면 ThreadLocal에 저장하여 재사용 * ThreadLocal을 검사하도록 수정 * refactor: business hour * refactor: BusinessHour를 값 타입으로 변경 * (shop): mybatis mapper 제거 * #93 카테고리 mybatis -> jpa (#108) * refactor: ShopCategoy 엔티티 추가 및 Category를 JPA로 * fix controller PathVariable error * ShopCategoryRepository moved into domain package * fix query typo * jmeter script * #96 jmeter script (#109) * jmeter script * util: jib 설정 추가, redis 설정 수정, docker-compose 수정 등 * util: jib 설정 추가, redis 설정 수정, docker-compose 수정 등 (#110) * import 한줄 제거 Co-authored-by: lyh7712 <mmj2rv9@kakao.com> * test: User가 없는 케이스 추가 refactor: Order 유효성 검사 Validator로 분리 validator 완료 * feat: Order의 Option 추가, Validator 추가 * feat: 주문총액 계산 * #107 장바구니 ArgumentResolver, 주문하기 컨트롤러 (#112) * test: User가 없는 케이스 추가 * feat: 컨트롤러에서 장바구니 정보를 파라미터로 받아온다. * chore: validation 수정, 테스트 수정 등 * Cart를 RequestBody로 받음. Cart 옵션 관련 클래스들 추가 * #85 배달 가능 지역 업데이트 API (#114) * feat: 배달가능지역 업데이트 기능 * refactor: 읍면동 조회 address mapper로 추상화 Co-authored-by: lyh7712 <mmj2rv9@kakao.com>
soongjamm
added a commit
that referenced
this pull request
Aug 13, 2021
* root package bluedelivery로 변경 * common.authentication 패키지 정리 * user package * api-application-domain-infra * config -> common.config * PasswordValidator를 controller와 같은 패키지로 이동 * test package * (AddressService) application -> domain 으로 이동 * category package * shop package * businesshour package * closingday package * menu 패키지 정리 * api(menu) 패키지 분할 * category adapter package * #101 @AuthenticationRequired + ArgumentResolver 이면 redis를 2번 조회하는 문제 (#102) * fix: 한 요청내에서 @AuthenticationRequired로 인증객체를 한번 조회하면 ThreadLocal에 저장하여 재사용 * ThreadLocal을 검사하도록 수정 * refactor: business hour * refactor: ShopCategoy 엔티티 추가 및 Category를 JPA로 * fix controller PathVariable error * ShopCategoryRepository moved into domain package * fix query typo * jmeter script * refactor: BusinessHour를 값 타입으로 변경 * (shop): mybatis mapper 제거 * #96 jmeter script (#109) * jmeter script * util: jib 설정 추가, redis 설정 수정, docker-compose 수정 등 * util: jib 설정 추가, redis 설정 수정, docker-compose 수정 등 (#110) * import 한줄 제거 * #107 주문 도메인 예외처리 (#111) * feat: 기본적인 주문 예외 발생 상황 * test: 예외 케이스 추가 * refactor: OrderTest * fix: MenuRepository JPA 상속 제거(아직 Menu Entity 사용 불가능) * #76 #93 영업시간 구현 수정, jpa 적용 (#106) * root package bluedelivery로 변경 * common.authentication 패키지 정리 * user package * api-application-domain-infra * config -> common.config * PasswordValidator를 controller와 같은 패키지로 이동 * test package * (AddressService) application -> domain 으로 이동 * category package * shop package * businesshour package * closingday package * menu 패키지 정리 * api(menu) 패키지 분할 * category adapter package * #101 @AuthenticationRequired + ArgumentResolver 이면 redis를 2번 조회하는 문제 (#102) * fix: 한 요청내에서 @AuthenticationRequired로 인증객체를 한번 조회하면 ThreadLocal에 저장하여 재사용 * ThreadLocal을 검사하도록 수정 * refactor: business hour * refactor: BusinessHour를 값 타입으로 변경 * (shop): mybatis mapper 제거 * #93 카테고리 mybatis -> jpa (#108) * refactor: ShopCategoy 엔티티 추가 및 Category를 JPA로 * fix controller PathVariable error * ShopCategoryRepository moved into domain package * fix query typo * jmeter script * #96 jmeter script (#109) * jmeter script * util: jib 설정 추가, redis 설정 수정, docker-compose 수정 등 * util: jib 설정 추가, redis 설정 수정, docker-compose 수정 등 (#110) * import 한줄 제거 Co-authored-by: lyh7712 <mmj2rv9@kakao.com> * test: User가 없는 케이스 추가 refactor: Order 유효성 검사 Validator로 분리 validator 완료 * feat: Order의 Option 추가, Validator 추가 * feat: 주문총액 계산 * #107 장바구니 ArgumentResolver, 주문하기 컨트롤러 (#112) * test: User가 없는 케이스 추가 * feat: 컨트롤러에서 장바구니 정보를 파라미터로 받아온다. * chore: validation 수정, 테스트 수정 등 * Cart를 RequestBody로 받음. Cart 옵션 관련 클래스들 추가 * #85 배달 가능 지역 업데이트 API (#114) * feat: 배달가능지역 업데이트 기능 * refactor: 읍면동 조회 address mapper로 추상화 Co-authored-by: lyh7712 <mmj2rv9@kakao.com> Co-authored-by: lyh7712 <mmj2rv9@kakao.com>
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
#93
빠르고 최소한의 변경을 목표로 했습니다 ..🙃ㅎㅎ