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

Add Woocomerce Onboarding task to set up Stripe #1492

Merged
merged 2 commits into from
Jul 22, 2024

Conversation

JulianBustamante
Copy link
Contributor

@JulianBustamante JulianBustamante commented Jul 17, 2024

Changes proposed in this Pull Request:

Closes https://github.com/Automattic/dotcom-forge/issues/8257

This PR adds the onboarding task to get paid with Stripe and removes Woo payments.

Screenshot 2024-07-18 at 10 59 07 PM

How to test the changes in this Pull Request:

  1. Create a free trial eCommerce site for Development following the steps described in this https://hostedwoo.wordpress.com/2023/02/09/setting-up-a-free-ecommerce-trial-site-for-development/
  2. Open atomic _cli and install Stripe plugin wp wpcomsh plugin use-managed woocommerce-gateway-stripe --activate
  3. Also, from the _cli Update the woocommerce_onboarding_profile option by using the following command wp option patch insert woocommerce_onboarding_profile partner stripe
  4. Check out this branch on your local environment and synchronize the files with the plugin on atomic by following these steps. https://teameaglep2.wordpress.com/how-to-guides/working-with-the-woocommerce-calypso-bridge/
  5. Go to the wc-admin page /wp-admin/admin.php?page=wc-admin
  6. Confirm that the Get paid with Stripe task is displayed.
  7. You should be redirected to Stripe Connect.

Other information:

  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you written new tests for your changes, as applicable?

FOR PR REVIEWER ONLY:

  • I have reviewed that everything is sanitized/escaped appropriately for any SQL or XSS injection possibilities. I made sure Linting is not ignored or disabled.

Copy link

github-actions bot commented Jul 17, 2024

Size Change: +71 B (+0.04%)

Total Size: 201 kB

Filename Size Change
./build/index.js 126 kB +71 B (+0.06%)
ℹ️ View Unchanged
Filename Size
./build/53.js 1.08 kB
./build/index.css 878 B
./build/marketing.js 58.3 kB
./build/payment-gateway-suggestions.css 1.24 kB
./build/payment-gateway-suggestions.js 6.57 kB
./build/plugins.js 3.93 kB
./build/style-index.css 2.15 kB
./build/style-marketing.css 800 B

compressed-size-action

@JulianBustamante JulianBustamante changed the title Add Wcommerce Onboarding task to set up Stripe Add Woocomerce Onboarding task to set up Stripe Jul 19, 2024
@JulianBustamante JulianBustamante marked this pull request as ready for review July 19, 2024 05:07
Copy link
Contributor

@epeicher epeicher left a comment

Choose a reason for hiding this comment

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

I have tested it, and it works as advertised. After following the test steps, I can see the Get paid with Stripe in the onboarding tasks. LGTM!

CleanShot 2024-07-19 at 13 19 14@2x

Copy link
Contributor

@moon0326 moon0326 left a comment

Choose a reason for hiding this comment

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

LGTM and tested well 👍 🚀

@JulianBustamante JulianBustamante merged commit 816a7e3 into master Jul 22, 2024
5 checks passed
@JulianBustamante JulianBustamante self-assigned this Jul 22, 2024
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.

4 participants