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

Experimental Stripe Connect test #2031

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Conversation

claudiulodro
Copy link
Contributor

This requires Automattic/newspack-blocks#1278 to work.

This PR adds a minimum possible Stripe Connect integration to the Simplified Donate Block. This was primarily so that we have a simple way to create purchases under a Stripe Connected account, for migration testing, but it also identifies most of the areas we'll need to update when we do add proper Stripe Connect support. This PR is definitely not intended to ever be merged.

To use

  1. Set up Stripe Connect on a Stripe account, so that it can be a "parent" account to other Stripe accounts.
  2. Connect a "child" Stripe account to the "parent" account (I have one already connected, ping me in Slack if you want access)
  3. At the Connect screen of the parent account, find the child account's account ID. This should start with acct_.
  4. In your wp-config add the following constant: define( 'NEWSPACK_STRIPE_CONNECTED_CHILD_ACCOUNT', 'acct_...' );
  5. In the Newspack Reader Revenue Stripe settings, use the API keys for the parent account, not the child account.
  6. Use the Simplified Donate Block. It should work nicely, and in the child account's payments/subscriptions you should see a 5% application fee added to the purchases:

Screen Shot 2022-09-22 at 11 47 57 AM

@yogeshbeniwal
Copy link
Contributor

@claudiulodro Interesting, May I know what use case you are targeting by adding integration to Stripe Connect.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants