Skip to content
This repository has been archived by the owner on Aug 13, 2022. It is now read-only.

#107 장바구니 ArgumentResolver, 주문하기 컨트롤러 #112

Merged
merged 7 commits into from
Aug 13, 2021

Conversation

soongjamm
Copy link
Collaborator

쿠키를 장바구니로 사용하고 ArgumentResolver를 이용해서 파라미터로 받아옵니다. (인증객체 처럼)

  • 쿠키에 장바구니 정보가 없으면 빈 장바구니 생성

@soongjamm soongjamm self-assigned this Aug 7, 2021
@soongjamm soongjamm changed the title #107 장바구니 ArgumentResolver, 주문하기 API #107 장바구니 ArgumentResolver, 주문하기 컨트롤러 Aug 7, 2021
@f-lab-bright
Copy link
Collaborator

@soongjamm @lyh7712
여기서 고민해봐야 할 건 장바구니를 쿠키로 전달하는 것과, POST의 body로 전달해서 주문을 실제로 요청할 때 처리하는 것 둘 중 어느게 더 좋을지 한 번 고민해봐도 좋을 것 같습니다.
일반 이커머스라면 쿠키로 구현하는게 괜찮은 선택일텐데, 배달 커머스에서도 동일한 장점을 가질 수 있을까요..?

* feat: 배달가능지역 업데이트 기능

* refactor: 읍면동 조회 address mapper로 추상화
@soongjamm soongjamm merged commit 37945a3 into #107-OrderDomainException Aug 13, 2021
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>
@soongjamm soongjamm deleted the #107-OrderApi branch September 11, 2021 17:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants