♻️ Refactor: Payment 결제 승인 관련 리팩토링 - [merged] #81

Closed GATBAWI closed 5 months ago

In GitLab by @oistmil on May 17, 2024, 01:35

Merges BE/refactor/payment -> develop

📄 제목

Payment 결제 승인 관련 리팩토링

✍ 작업 내용 [FE/BE]

Payment와 PaymentHistory 엔티티를 구분하여 결제 주기를 자세히 관리하고 오류 추적이 용이할 수 있도록 변경
결제 state 세분화
결제 성공/실패에 대한 response dto 통일
토스페이먼츠 API 호출 후 받는 dto 분리
추후 결제 재시도와 장부 업데이트를 위한 컬럼 추가

MR 타입

[x] 기능 추가
[ ] 기능 수정
[ ] 기능 삭제
[ ] 버그 수정
[x] 코드 리팩토링
[ ] UI 추가 및 변경

반영 브랜치

BE/refactor/payment -> develop

테스트 결과

[x] Swagger 테스트 결과 이상 없습니다.
[x] local ci 테스트 결과 이상 없습니다.
[x] Postman 테스트 결과 이상 없습니다.

In GitLab by @shonee99 on May 17, 2024, 10:58

Commented on Backend/src/main/java/com/homegravity/Odi/domain/payment/controller/PaymentController.java line 49

성공 실패 합치셧군요 좋슴다

In GitLab by @shonee99 on May 17, 2024, 11:00

Commented on Backend/src/main/java/com/homegravity/Odi/domain/payment/dto/toss/PSPConfirmationResponseDto.java line 26

이 NORMAL은 무슨뜻인가욤

In GitLab by @soomni20 on May 17, 2024, 11:04

Commented on Backend/src/main/java/com/homegravity/Odi/domain/payment/dto/response/PaymentConfirmationResponseDto.java line 30

from 메소드 아닌가용?

In GitLab by @soomni20 on May 17, 2024, 11:08

Commented on Backend/src/main/java/com/homegravity/Odi/domain/payment/dto/toss/PaymentDetails.java line 19

이 친구는 private 해주면 좋을 지도?

In GitLab by @soomni20 on May 17, 2024, 11:10

Commented on Backend/src/main/java/com/homegravity/Odi/domain/payment/service/PaymentService.java line 65

이뻐용 이렇게 쓸 수 있다니 너무 신기해요

In GitLab by @soomni20 on May 17, 2024, 11:12

approved this merge request

In GitLab by @soomni20 on May 17, 2024, 11:12

감사합니당 고생하셨어요

In GitLab by @shonee99 on May 17, 2024, 11:13

approved this merge request

In GitLab by @oistmil on May 17, 2024, 11:16

Commented on Backend/src/main/java/com/homegravity/Odi/domain/payment/controller/PaymentController.java line 49

마자용~!~!

In GitLab by @oistmil on May 17, 2024, 11:17

Commented on Backend/src/main/java/com/homegravity/Odi/domain/payment/dto/toss/PSPConfirmationResponseDto.java line 26

결제 방식에 일반결제, 자동결제, 브랜드페이로 나뉘는데 저희는 일단 되는게 일반결제밖에 없어서 표시하려고 써뒀습니다..

In GitLab by @oistmil on May 17, 2024, 11:17

Commented on Backend/src/main/java/com/homegravity/Odi/domain/payment/dto/response/PaymentConfirmationResponseDto.java line 30

마자용 그런데 아래에 있는 of 메서드랑 역할이 완전 똑같아서 일단 그대로 썼어요. 좋은 이름을 고민해봐야겠습니다..!

In GitLab by @oistmil on May 17, 2024, 11:18

Commented on Backend/src/main/java/com/homegravity/Odi/domain/payment/dto/toss/PaymentDetails.java line 19

얘는 리팩토링 다시해서 가져오겠습니다.

In GitLab by @oistmil on May 17, 2024, 11:29

Commented on Backend/src/main/java/com/homegravity/Odi/domain/payment/dto/toss/PaymentDetails.java line 19

changed this line in version 2 of the diff

In GitLab by @oistmil on May 17, 2024, 11:29

added 1 commit

6ad81fd3 - ♻️ Refactor: PaymentDetails와 PaymentFailure 내부 클래스로 변경

Compare with previous version

In GitLab by @oistmil on May 17, 2024, 11:30

Commented on Backend/src/main/java/com/homegravity/Odi/domain/payment/dto/toss/PaymentDetails.java line 19

리팩토링 완료!

In GitLab by @oistmil on May 17, 2024, 13:27

mentioned in commit 4637b4b53586ce504d223414eab11ee9ef416069

Gravity-of-home / ODI