-
Notifications
You must be signed in to change notification settings - Fork 19
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
Feature request: ability to set different return and cancel urls #13
Comments
Can you elaborate why this would be necessary? You can check if return was successful with Of course this can be done, but would be breaking change. If you can provide real use case for this, I can implement this. |
i could do this, but it would be much easier to have that parameter, get rid of one redirect. Maybe it is not worth of it, but it would be nice. Just sharing my opinion :) |
Sharing opinions is always welcomed. Thanks! I just thought that it has some certain case where you can't use this library at all if that's not implemented. |
Actually, I came up with workaround for you if you still want different URLs. Since you can override request data, you can also override
|
Thanks, i will try :) |
Expected Behavior
Return and cancel url should not be same, at least there should be option to change it
Actual Behavior
Request url is used as RETURN and CANCEL
Steps to Reproduce the Problem
The text was updated successfully, but these errors were encountered: