Release 210721 #227
Merged
Release 210721 #227
Conversation
This file contains hidden or 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
기존에는 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 형태로 저장합니다.
[feat][STORE-1431] Payment 하위에 Charge 모델 추가
1. Charge에 status 추가 2. payment모델안의 상수명 복수형 반영
기존의 supplement를 하나를 두고 운영하는 방안은 결제 process의 흐름 안에서 charge의 다양한 정보 변화를 다 덮어써야 한다는 문제가 있었습니다. 따라서 payment가 charge 자체를 process의 흐름에 따라 복수개를 가지도록, 그리고 그 charge가 가지는 정보의 의미가 명확해 지도록 컬럼명을 수정했습니다. 그에 따라 알맞는 히스토리의 charge가 생성되도록 data migration도 수정하였습니다.
paid가 nil인 주문이 있어서 마이그레이션이 불가한 것을 고치기 위한 코드입니다. 프로덕션 환경에서 해당하는 레코드는 1건입니다. id: 4379
스토어측에서 변경된 charge의 schema에 따라 charge생성 로직을 보완했습니다. charge id를 통해 omise의 api에서 charge를 가져오고 데이터를 상황에 맞게 생성합니다. 기존 charge_id가 비정상적이라서 charge를 가져오지 못한다면 가지고있는 데이터를 조합해 생성합니다. credential파일에 omise와 관련된 key를 입력해 저장했습니다. staging 서버용 RDS의 엔드포인트 변경사항을 반영했습니다.
Feat/integrate vn payment gateway
[feat]결제 모듈 확장 및 안정화
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.
결제 모듈 확장 및 안정화
목적 및 요약
기타