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

Payment method duplicates warning still thrown despite disabling the Stripe gateway #9271

Closed
pierorocca opened this issue Aug 16, 2024 · 3 comments · Fixed by #9663
Closed
Assignees
Labels
focus: checkout payments 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

@pierorocca
Copy link
Contributor

Describe the bug

I had the WooPayments and Stripe gateways installed, activated, and enabled. Both had Apple Pay and Google Pay enabled. Doing so displayed the inline notification warning in WooPayments settings as expected.

I then disabled the Stripe gateway, without first disabling Apple Pay and Google Pay, and the error still persists. Though the warning is dismissible, the logic should if possible be aware that the gateway has been disabled and hide the warning.

image

@pierorocca pierorocca added type: bug The issue is a confirmed bug. focus: checkout payments priority: medium The issue/PR is medium priority—non-critical functionality loss, minimal effect on usability labels Aug 16, 2024
@mdmoore mdmoore self-assigned this Oct 31, 2024
@mdmoore
Copy link
Member

mdmoore commented Oct 31, 2024

👋 @pierorocca @Automattic/heisenberg - I haven't been able to reproduce this one. I've activated WooCommerce Stripe Gateway alongside WooPayments and enabled Apple/Google Pay in both plugins. I get the duplication notice when both plugins are active, but when deactivating WooCommerce Stripe Gateway (without first disabling Apple/Google Pay), the duplication notice disappears. Looking at the Duplicates_Detection_Service class, each check (CC, APM, Payment Requests) first checks for enabled gateways or registered payment methods. Those checks should be covering this since inactive plugins wouldn't have registered gateways. Can anyone still reproduce this? Here's what I'm getting:

Screen.Capture.on.2024-10-31.at.14-38-01.mp4

@pierorocca
Copy link
Contributor Author

Hey @mdmoore instead of deactivating the plugin, try disabling the gateway through the toggle in WooPayments settings.

@mdmoore
Copy link
Member

mdmoore commented Nov 4, 2024

Thanks for clarifying, @pierorocca. That's clear in your description. Not sure why I was stuck on deactivating the plugin 🤦‍♂️. I was able to reproduce the issue when disabling the gateway. Solved with #9663.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
focus: checkout payments 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
2 participants