-
-
Notifications
You must be signed in to change notification settings - Fork 25
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
Configurable possibility to channel on GoPay #47
Conversation
Now you can configure (changeChannel config parameter) a possibility to allow or deny a change of a payment channel on GoPay gate.
I like it, but only as property and settter. Could you change it please? |
Do you think without setting up from config? |
@martinsifra I could be over config, but via setter method. For example like this. if ($config[...]) {
$class->addSetup('setChangeChannel', TRUE);
} A could you merge to one commit please? |
Yeah, I see. I'll make commit today. |
ping @martinsifra |
Sorry, I know. It is on my schedule. |
@@ -30,6 +30,7 @@ class Extension extends CompilerExtension | |||
'gopayId' => NULL, | |||
'gopaySecretKey' => NULL, | |||
'testMode' => TRUE, | |||
'changeChannel' => TRUE, |
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.
better default NULL
@martinsifra It's great. Could you prepare simple test? And primary, rebase it to 1 commit? |
Thanks, I could, but I've never do that. So I'll need more time :-) |
@martinsifra It's oukey. I'll do it by my own. |
Merged via d545d5d. Thank you. |
Now you can configure (changeChannel config parameter) a possibility to
allow or deny a change of a payment channel on GoPay gate.