-
Notifications
You must be signed in to change notification settings - Fork 1.8k
feat: PayPal env routing based on settings #3479
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
Conversation
@uds5501 @shreyanshdwivedi @CosmicCoder96 I've checked in this locally and can confirm that it hits the sanbox API for test mode and live API for live mode. Please do confirm this. |
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.
Please fix CI
ae8e916
to
ba74acb
Compare
0aea579
to
5f78d76
Compare
5f78d76
to
2f09717
Compare
@CosmicCoder96 The paypal button requires the environment parameter to be |
Fixes #3413
Short description of what this resolves:
Environment is being forced to a live environment without any param specified. TO properly route the API hit to sanbox/live, settings value is being passed.