Skip to content
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

PayPal REST #3222

Open
wants to merge 18 commits into
base: dev
Choose a base branch
from
Open

PayPal REST #3222

wants to merge 18 commits into from

Conversation

dparker1005
Copy link
Member

This work is not ready for production environments.

@dparker1005 dparker1005 added this to the 3.4 milestone Dec 17, 2024
@dparker1005
Copy link
Member Author

At this point, the gateway feels relatively stable but is still not ready for a full release. This PR hides the gateway behind a constant that needs to be set with the following code:
define( 'PMPRO_PAYPALREST_BETA', true );

There is a known incompatibility with the PMPro Pay By Check Add On (credit card fields showing for PayPal REST) that will need to be resolved in PBC or by core handling multiple gateways natively. Similar incompatibilities exist with the Add PayPal Express Add On, though that Add On doesn't really make sense to use alongside PayPal REST.

There is also an open question around how REST API credentials should be disabled. Right now, the "disconnect" option just deletes the keys from the database but they could still be used elsewhere.

On our oauth server, we also still need to add PayPal Multiparty keys for the Stranger Studios platform account.

@dparker1005 dparker1005 marked this pull request as ready for review January 10, 2025 17:36
@dparker1005
Copy link
Member Author

Also important to note:

@dparker1005 dparker1005 changed the title Work In Progress - PayPal REST PayPal REST Jan 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant