issues
search
Picktartup
/
Pickstartup-walletservice
0
stars
0
forks
source link
[fix]: 계약관련로직 contractservice 로 이관
#15
Closed
RyuChaeHyun
closed
6 days ago
RyuChaeHyun
commented
6 days ago
👀 관련 이슈
12
✨ 작업한 내용
TokenTransaction
클래스에서
paymentId
필드 제거
paymentId
필드가 더 이상 필요하지 않아 해당 필드를 제거하여 코드 개선
스마트 컨트랙트 계약 로직을
ContractService
로 이관
스마트 컨트랙트 관련 계약 로직을
ContractService
클래스에 분리하여 MSA 유지보수 용이성 증가
🌀 PR Point
코드 리뷰가 필요한 부분:
TokenTransaction
에서
paymentId
를 제거한 후 관련 기능이 정상적으로 작동하는지 확인
ContractService
로 로직을 이관한 후, 기존 스마트 컨트랙트 관련 기능들이 의도대로 동작하는지 검토
🍰 참고사항
ContractService
로의 이관 작업은 스마트 컨트랙트 관련 기능이 앞으로 확장될 가능성을 고려하여 분리한 작업입니다.
이전 코드에서
paymentId
를 사용하는 부분을 모두 점검하고 수정했는지 확인이 필요합니다.
na-rong
commented
6 days ago
LGTM! 수고하셨어용~
👀 관련 이슈
12
✨ 작업한 내용
TokenTransaction
클래스에서paymentId
필드 제거paymentId
필드가 더 이상 필요하지 않아 해당 필드를 제거하여 코드 개선ContractService
로 이관ContractService
클래스에 분리하여 MSA 유지보수 용이성 증가🌀 PR Point
TokenTransaction
에서paymentId
를 제거한 후 관련 기능이 정상적으로 작동하는지 확인ContractService
로 로직을 이관한 후, 기존 스마트 컨트랙트 관련 기능들이 의도대로 동작하는지 검토🍰 참고사항
ContractService
로의 이관 작업은 스마트 컨트랙트 관련 기능이 앞으로 확장될 가능성을 고려하여 분리한 작업입니다.paymentId
를 사용하는 부분을 모두 점검하고 수정했는지 확인이 필요합니다.