Skip to content

Conversation

@heozeop
Copy link
Contributor

@heozeop heozeop commented Dec 1, 2020

목적

주문 당시 상품 가격에 대한 우려가 있어 해당 요소를 주문 정보에서 가져온다.

작업

  1. adapter
    채널 별로 데이터 가져옴
  2. Saver
    저장 로직에서 데이터 저장함
  3. model
    unit price 컬럼 추가

외부채널 cart item에 값 보정을 위한 unit price 컬럼 추가 및 데이터 받아옴
…o feat/add_unit_price_2_external_channel
@heozeop heozeop requested a review from PLUX-KR December 1, 2020 03:29
@heozeop heozeop merged commit 9d40cb8 into feat/dump_haravan_orders Dec 1, 2020
@heozeop heozeop deleted the feat/add_unit_price branch December 1, 2020 11:57
PLUX-KR added a commit that referenced this pull request Dec 2, 2020
… 도입 (#125)

* 핫픽스: 배송 관련 필드 타입 수정 (#49)

* shipping_addresses 테이블 문자셋을 utf8mb4 로 변경

* ship_info.user_memo 필드 타입을 Text 로 변경

* 핫픽스 : gomibranch 에 store module 명시

* 수정 : gomibranch payment manager 추가

* 수정: 마이그레이션으로 인한 annotate 작동

* 수정: 외부채널 컨트롤러가 매니저에게 전달하는 파라미터를 해시로 변경

* 수정: 베이스 어댑터의 parse_query_hash 메소드 수정

* 수정: 하라반, 티키 어댑터에 변경사항 적용

* 수정: 베이스 어댑터에 주석 추가

* 수정: 개발 환경에서 사용할 lazada app console 정보 업데이트

* 수정: 하라반 어댑터 오류 수정

* 수정: 라자다 어댑터에 parse_query_hash 메소드 업데이트 적용

* 수정: 파트너 센터 유저의 jwt 기한을 임시로 1년으로 수정

* 수정: cors에 스테이징 파트너센터 url을 추가

* 수정: 쇼피 어댑터 일부 수정, 승찬님께 전달

* 형식: uniprice 추가에 따른 주석 변경

* 리팩토링: 쇼피 어뎁터 parse query방식 수정

사용하기로 협의한대로 사용할 수 있도록 parse query 변경

* 기능: external_channel_cart_item에 unit price 컬럼 추가 (#124)

* 기능: 외부채널 cart item에 unit 가격 추가

외부채널 cart item에 값 보정을 위한 unit price 컬럼 추가 및 데이터 받아옴

* 형식: DB 변경에 따른 주석 변경

* 리팩토링: sendo 어뎁터 요청 방식 규정

sendo 어뎁터에서 새로 규정한 request 방식으로 요청을 보내고 받을 수 있도록 리팩토링

* 리팩토링: 쇼피 호출 로직 변경

기존에 2중 while문 사용하면서 읽기 어려워서 query생성하는 함수를 추가했다.

* 수정: 라자다 어댑터에서 데이터가 없을 경우 에러를 발생시키지 않도록 수정

* 수정: 용현님 코드리뷰 적용

* 수정: 외부채널 어댑터 query_hash 데이터 스타일 통일

* 버그: 쇼피 어뎁터 일자 안잘리는 문제

hash 포인터를 받아서 생기는 에러였다.

Co-authored-by: Yonghyun Kim (Freddy) <fred@gomicorp.com>
Co-authored-by: Lee seung chan <dltmdcks702@gmail.com>
Co-authored-by: Bran <40688044+june20516@users.noreply.github.com>
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