-
Notifications
You must be signed in to change notification settings - Fork 0
[#24] 게임 결제 로직 #61
base: develop
Are you sure you want to change the base?
[#24] 게임 결제 로직 #61
Conversation
현재 로직의 뼈대만 잡은 상태로 이후 세부 로직과 단위 테스트를 추가할 예정
@@ -74,4 +77,10 @@ public void deleteUser(@PathVariable long id) { | |||
|
|||
userService.deleteUser(id); | |||
} | |||
|
|||
@GetMapping("/{id}/games") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
구매목록이니 games보단 purchases가 낫지 않을까요?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
네, 25번 브랜치 머지 후 수정하겠습니다.
@@ -28,5 +29,33 @@ pipeline { | |||
} | |||
} | |||
} | |||
|
|||
stage('Deploy') { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
작업내용이 좀 섞여있는 것 같은데 브랜치 확인 부탁드려요~
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Jenkins CD 관련 브랜치를 머지한 후에 수정할 점을 발견해서 작업 내용이 섞인 것 같습니다. 이 부분은 새 브랜치를 열어서 다시 머지 요청 받겠습니다.
@AllArgsConstructor | ||
public class AccountDto { | ||
|
||
long id; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
접근제어자는 명시적으로 붙여주시는게 좋을 것 같네요~
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
네, 지금 보니 다른 DTO 클래스도 접근 제어자를 붙여야 겠네요. 이것도 다른 브랜치에 머지 요청 받겠습니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
우선 현 브랜치와 관계가 있는 AccountDto와 PurchaseDto에만 리뷰 반영하겠습니다.
public class MobileChargeService implements ChargeService { | ||
|
||
@Override | ||
public void chargeCash(long userId, int amount) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
여기는 가짜 결제메소드인건가요?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
결제 방법에 따라 메서드를 만들 예정이었습니다. 지금은 mobile 결제 방법의 클래스만 만든 상태이고요.
This reverts commit 968a0e9
develop 브랜치에만 배포되도록 막았던 코드를 revert하면서 Jenkins 커밋이 불발돼 위 실패가 떴습니다. 현 브랜치 코드의 빌드 및 테스트는 정상 작동 확인했습니다. |
작업 계속 진행해주시면 될 것 같네요~ |
Jenkinsfile 추가 수정
[#27] DTO 클래스 수정
iamport를 연동해봐도 좋을 것 같네요~ |
네, 적용해보겠습니다. |
올리신 커밋에는 아임포트 API를 통한 결제 무결성 검증 등이 빠져있는 것 같아요~ 도큐먼트를 정독하고 모두 구현이 되면 좋을 것 같아요 |
게임 결제를 위한 로직
iamport API
를 통해 결제를 하면 해당 게임의 정보와 결제 정보를 파라미터로 넘겨 저장.결제는 카카오와 네이버만 있다는 가정 하에 구현