Closed khj309 closed 2 years ago
composer 저장소에도 배포해야 하는데 Sementic versioning으로 태깅해주시고 releases 해주시면 되요
배포 확인은 https://packagist.org/packages/iamport/rest-client
여기서 가능합니다.
테스트 작성도 optional 필드가 생긴만큼 기존의 delete_subscribe_customer
외에도 추가된 필드로 테스트 메소드가 하나 더 추가되면 좋을 것 같습니다.
🌏 iamport-rest-client-php PR 🌏
PR 요약
reason
,extra[requester]
파라미터(Optional)를 지원하도록 수정하였습니다.변경된 부분 👍
$reason
,$extra
멤버 필드를 추가하여 delete 메서드에서 api 요청 시 전달하도록 수정, 그 외 메서드에서 추가된 필드에 영향을 받지 않도록 수정하였습니다.extra[]
파라미터를 지원하기 위한 SubscribeCustomerExtra 클래스가 추가되었습니다.영향을 받는 다른 컴포넌트 🕸
None
집중해서 리뷰할 곳 💡
$reason
,$extra
멤버 필드를 추가하여 delete 메서드 및 그 외 다른 모든 메서드에 영향이 있습니다.extra[]
파라미터를 지원하기 위해 SubscribeCustomerExtra 클래스를 추가했습니다. 해당 클래스의 네이밍을 좀 고민했는데 혹시 더 나은 방법이나 이름이 있다면 의견 부탁드립니다.테스트 ⚒
composer test
통과 확인참고링크 🎫
Due Date: 2022-05-09, 사실 ASAP 입니다