-
Notifications
You must be signed in to change notification settings - Fork 11
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
GMO後払い専用のGatewayを、全てのリンク決済で利用できるGatewayに置き換え #110
Conversation
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.
LGTMです!
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.
よさそうです 🙆
README.md
Outdated
@@ -302,12 +302,12 @@ else | |||
end | |||
``` | |||
|
|||
### GMO Payment After Delivery | |||
### GMO Link Payment Delivery |
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.
「Delivery」不要かも?と思います
「GMO後払い」の英語名が「GMO Payment After Delivery」だったので「Delivery」ついていた背景だったと思うので
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.
「Link Payment」がリンクタイプ決済の総称である(このGatewayがすべてのリンクタイプの決済で使える)ことがわかるように補足があっても親切そうですね(急いでマージした方がよければ後から追加でも)
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.
「Link Payment」がリンクタイプ決済の総称である(このGatewayがすべてのリンクタイプの決済で使える)ことがわかるように補足があっても親切そうですね
EpsilosLinkPaymentGateway is available in all link payments.
For example, GMO Payment After Delivery.
と書いておきますね。
READMEにコメントしましたが、実装の方はよさそうです 🙆 |
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.
対応ありがとうございますー!LGTMです 🙆
レビューありがとうございます! |
やったこと
EpsilonGmoAfterGateway
を削除EpsilonLinkPaymentGateway
を追加st_code
は呼び出し側で指定した値を使うように修正このPRによって変わること
GMO後払い決済を行うときの呼び出し方が変わります。
変更前
変更後
参考:
イプシロン決済システム仕様書 Ver.1.0.41 p44, 45