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

WooCommerce Stripe #75

Merged
merged 2 commits into from
Dec 14, 2023
Merged

WooCommerce Stripe #75

merged 2 commits into from
Dec 14, 2023

Conversation

bd-viget
Copy link
Contributor

@bd-viget bd-viget commented Dec 13, 2023

Summary

This PR adds the WooCommerce Stripe extension and updates WooCommerce to the latest version (8.4.0).

Issues

Testing Instructions

  1. Create a new site
  2. Visit /wp-admin/admin.php?page=wc-settings&tab=checkout&section=stripe
  3. Proceed with the configuration of Stripe for that site
  4. Test credentials can be found in 1Password.

Screenshots

Screenshot 2023-12-13 at 3 29 24 PM

Note: There is a bug in this branch, which is fixed in another branch, that throws an error when visiting the new site for the first time. It can be ignored for the sake of testing this issue.

@clatwell
Copy link
Contributor

@bd-viget - I'm curious if we (or GB) should create a "Test Nonprofit" Stripe merchant account that we can use for testing purposes. I can take the lead on that tomorrow. IIRC from the Rendr project, this is pretty easy / free to set up and you can get pretty far before activating the account.

@clatwell
Copy link
Contributor

Update: I spun up Stripe account and added login creds to 1Pass.

Once this deploys to Staging, I'll go through the motions of connecting the account and see how far we can get with a free Stripe account / no bank info / account in test mode.

If we need to go further with our test account to be able to test placing Bids, we can coordinate with GB.

cc @shascher

Copy link
Contributor

@nathan-schmidt-viget nathan-schmidt-viget left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@bd-viget bd-viget merged commit c4fd7dc into main Dec 14, 2023
1 check passed
@bd-viget bd-viget deleted the bd/22-woocommerce-stripe branch December 14, 2023 16:44
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.

3 participants