Thanks to visit codestin.com
Credit goes to github.com

Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: gomicorp/EcommerceAPI
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: develop
Choose a base ref
...
head repository: gomicorp/EcommerceAPI
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: master
Choose a head ref
Checking mergeability… Don’t worry, you can still create the pull request.
  • 5 commits
  • 19 files changed
  • 3 contributors

Commits on May 20, 2021

  1. Merge pull request #217 from gomicorp/develop

    Release210520
    june20516 authored May 20, 2021
    Configuration menu
    Copy the full SHA
    51ea382 View commit details
    Browse the repository at this point in the history

Commits on Jul 14, 2021

  1. Configuration menu
    Copy the full SHA
    4b20bd5 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    ae9ba74 View commit details
    Browse the repository at this point in the history
  3. Merge pull request #224 from gomicorp/hotfix/cl_backoffice_open

    [hotfix][STORE-1413] 칠레 백오피스 생성을 위한 데이터 준비
    aiody authored Jul 14, 2021
    Configuration menu
    Copy the full SHA
    a492919 View commit details
    Browse the repository at this point in the history

Commits on Jul 21, 2021

  1. Release 210721 (#227)

    * [feat][STORE-1431] Payment 하위에 Charge 모델 추가
    
    기존에는 PG의 결제 정보를, key를 통해 필요시마다 api로 얻어왔었습니다.
    그러나 다른 PG 연동을 확장함에 따라, api를 상시 제공 하지 않고, event 발생 전후에만 데이터를 보내주는 PG들도 대응할 필요가 있었습니다. 따라서, 필요시에 접근할 수 있도록 모델을 만들어 해당 리소스를 저장해야 합니다.
    
    이를 위해,
    1. payment를 country구분하였습니다. [STORE-1379]
    2. payment의 속성인 pay method의 card 항목을 pg로 세분화 했습니다. [STORE-1378]
    3. payment 하위에 charge 모델을 만들었습니다.[STORE-1431]
    3-1. charge 모델은, 여러 pg의 response에 유연하게 대응하기 위하여, charge를 식별할 수 있는 필수 정보만을 컬럼으로 갖고, 디테일한 내용은 supplement 컬럼에 json 형태로 저장합니다.
    
    * [refactor][#221] payment의 상수에 공백 추가
    
    * [feature][STORE-1431] Charge에 status 추가
    
    1. Charge에 status 추가
    2. payment모델안의 상수명 복수형 반영
    
    * [feature][STORE-1431] Charge 스키마 및 활용 방안 수정
    
    기존의 supplement를 하나를 두고 운영하는 방안은 결제 process의 흐름 안에서 charge의 다양한 정보 변화를 다 덮어써야 한다는 문제가 있었습니다. 따라서 payment가 charge 자체를 process의 흐름에 따라 복수개를 가지도록, 그리고 그 charge가 가지는 정보의 의미가 명확해 지도록 컬럼명을 수정했습니다.
    그에 따라 알맞는 히스토리의 charge가 생성되도록 data migration도 수정하였습니다.
    
    * [refactor] 현재 payment에 charge 모델을 설정하는 마이그레이션 파일 수정
    
    paid가 nil인 주문이 있어서 마이그레이션이 불가한 것을 고치기 위한 코드입니다.
    프로덕션 환경에서 해당하는 레코드는 1건입니다. id: 4379
    
    * [docs] 마이그레이션으로 인한 주석 변경
    
    * [feat] current payment에 대한 charge 생성 datamigration 보완
    
    스토어측에서 변경된 charge의 schema에 따라 charge생성 로직을 보완했습니다.
    charge id를 통해 omise의 api에서 charge를 가져오고 데이터를 상황에 맞게 생성합니다.
    기존 charge_id가 비정상적이라서 charge를 가져오지 못한다면 가지고있는 데이터를 조합해 생성합니다.
    
    credential파일에 omise와 관련된 key를 입력해 저장했습니다.
    staging 서버용 RDS의 엔드포인트 변경사항을 반영했습니다.
    
    * [refactor] datamigration이 하나의 transaction으로 묶이도록 변경
    
    * [refactor]omise 리퀘스트 거절시 재시도 로직 작성
    
    * [hotfix] gemfile의 'devise' 젬 브랜치 변경
    
    * [refactor]테스트 주석 삭제
    
    Co-authored-by: Robin Kim <[email protected]>
    Co-authored-by: lucas <[email protected]>
    3 people authored Jul 21, 2021
    Configuration menu
    Copy the full SHA
    6688e7e View commit details
    Browse the repository at this point in the history
Loading