Fix/add parameter support cancel payment #11
Conversation
khj309
requested review from
smc0210,
moonjihae,
aimpugn,
SoleeChoi and
SonnySon17
|
change log์ ํจํค์ง ๋ฒ์ ์ ๋ฆด๋ฆฌ์ฆ์ ๋ณ๊ฒฝํ๊ฒ ์ต๋๋ค. |
example/payment/cancel.php
Outdated
| @@ -39,6 +41,14 @@ | |||
| $request->refund_holder = 'ํ๋ถ๋ ๊ฐ์๊ณ์ข ์๊ธ์ฃผ'; | |||
| $request->refund_bank = 'ํ๋ถ๋ ๊ฐ์๊ณ์ข ์ํ์ฝ๋'; | |||
| $request->refund_account = 'ํ๋ถ๋ ๊ฐ์๊ณ์ข ๋ฒํธ'; | |||
|
|
|||
| $extra = new CancelPaymentExtra(); | |||
| if ( in_array($extra_requester, ['admin', 'customer'])) { | |||
There was a problem hiding this comment.
extra_requester๋ผ๋๊ฒ admin ์๋๋ฉด customer๋ง ๊ฐ๋ฅํ๊ฑฐ๋ฉด Enum( ์ ํํ๋ Enum์ ์๋์ง๋ง Enum์ค๋ฝ๊ฒ ๋ง๋ค์ด ๋์ /src/Enum ) ์ฒ๋ฆฌ ํ๋๊ฑด ์ด๋ค๊ฐ์?
There was a problem hiding this comment.
์ฐธ๊ณ ๋ก ์ด ๊ฐ์ด ์ ํํ ์ด๋ค๊ฑด์ง ์ ๊ฐ ๋ชฐ๋ผ์ ๊ทธ๋ฐ๋ฐ pg ์ฌ๋ณ๋ก ํด๋น ๊ฐ์ optional ์ฌ๋ถ๊ฐ ๋ค๋ฅด๊ณ value ํน์ ์ด ํ๋ค๋ค๋ฉด ์ง๊ธ์ด ๋ ๋์๊ฒ ๊ฐ์ต๋๋ค.
There was a problem hiding this comment.
์์ํฌํธ API ๋ฌธ์์์ ํด๋น ํ๋ผ๋ฏธํฐ ์ฌ์์ด ์๋ด๋์ด ์์ง ์๊ณ ๋ค์ด๋ฒ์ฐ๋๊ฐ์ด๋์ ์๋ด๋์ด ์๋๊ฑธ๋ก ๋ณด์ ๋ค์ด๋ฒํ์ด ์ ์ฉ์ผ๋ก ํ๋จ๋ฉ๋๋ค.
๊ทธ๋ฆฌ๊ณ ๋ค์ด๋ฒํ์ด ์ธก์์๋ ํ๋ผ๋งํฐ ๊ฐ์ 1(customer), 2(admin)๋ก ํน์ ํ ์ ์์ด์ ์๊ฒฌ ์ฃผ์ enum์ผ๋ก ์ฒ๋ฆฌํ๋๊ฒ ๋ ์ข์ ๊ฒ ๊ฐ์ต๋๋ค.
There was a problem hiding this comment.
๋ค์ด๋ฒํ์ด ์ ์ฉ ํ๋ผ๋ฏธํฐ๋ก CancelPaymentRequester.php enum ์ถ๊ฐ ํ์ต๋๋ค.
๐ iamport-rest-client-php PR ๐
PR ์์ฝ
refund_tel,extra[requester]ํ๋ผ๋ฏธํฐ๋ฅผ ์ง์ํ๋๋ก ์์ ํ์์ต๋๋ค.๋ณ๊ฒฝ๋ ๋ถ๋ถ ๐
CancelPaymentํด๋์ค์$refund_tel,$extra๋ฉค๋ฒ ์ถ๊ฐextra[]ํ๋ผ๋ฏธํฐ๋ฅผ ์ง์ํ๊ธฐ ์ํCancelPaymentExtraํด๋์ค๊ฐ ์ถ๊ฐ๋์์ต๋๋ค.์ํฅ์ ๋ฐ๋ ๋ค๋ฅธ ์ปดํฌ๋ํธ ๐ธ
None
์ง์คํด์ ๋ฆฌ๋ทฐํ ๊ณณ ๐ก
ํ ์คํธ โ
composer testํต๊ณผ ํ์ธ์ฐธ๊ณ ๋งํฌ ๐ซ
Due Date: 2022-05-16