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

WooPayments status and buttons are incorrect on Payments settings page when Stripe onboarding is started but not completed #8813

Open
timmy5685 opened this issue May 10, 2024 · 1 comment
Assignees
Labels
category: core WC Payments core related issues, where it’s obvious. focus: account lifecycle priority: medium The issue/PR is medium priority—non-critical functionality loss, minimal effect on usability type: bug The issue is a confirmed bug.

Comments

@timmy5685
Copy link

Describe the bug

If a merchant reaches the first screen of stripe onboarding but exits, we display the toggle for WooPayments on the Payments settings screen as enabled and the button as "Manage" which is incorrect. It appears that we are not taking into account whether or not Stripe onboarding steps were completed by the merchant - we just assume after the first stripe onboarding screen is reached that the merchant has completed it.

To Reproduce

  1. Install WooPayments
  2. Complete MOX and go to first step of Stripe onboarding.
  3. Click "Back to WooPayments"
  4. Go to WooCommerce > Settings > Payments
  5. WooPayments status and button are incorrect in Payment Methods table.

Actual behavior

WooPayments shows as enabled on payment settings screen and button says "Manage" even though the merchant did not complete the onboarding flow.

Screenshots

Screenshot 2024-05-10 at 3 03 25 PM

Expected behavior

WooPayments shows as disabled on payment settings screen and the button says "Finish set up" until the merchant completes the stripe onboarding flow.

@timmy5685 timmy5685 added type: bug The issue is a confirmed bug. category: core WC Payments core related issues, where it’s obvious. priority: medium The issue/PR is medium priority—non-critical functionality loss, minimal effect on usability focus: account lifecycle labels May 10, 2024
@aheckler
Copy link
Member

My two cents:

I agree that this is suboptimal and should not happen, but I'm not sure if this issue is specific to WooPayments. For example, if I have a brand new Woo store and I click the toggle to enable the Direct Bank Transfer payment method, it gets enabled (and the button changes to say "Manage" after a page refresh) even though I haven't added my bank account info to it yet.

Same with the Stripe extension: if I install it and go a step or two into the onboarding flow, then return to my site, I see that it's enabled even though I never finished onboarding and cannot take payments or anything with it.

Screenshot taken on 2024-05-13 at 10 14 52 UTC@2x

Screenshot taken on 2024-05-13 at 10 12 47 UTC@2x

It may be that the larger issue here is that core should somehow be detecting when payment methods are ready to actually take payment vs. just being enabled. That seems like a larger discussion though.

@oaratovskyi oaratovskyi self-assigned this May 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: core WC Payments core related issues, where it’s obvious. focus: account lifecycle priority: medium The issue/PR is medium priority—non-critical functionality loss, minimal effect on usability type: bug The issue is a confirmed bug.
Projects
None yet
Development

No branches or pull requests

3 participants