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

Mollie Plugin doesn't check for eligibility #2973

Closed
martijnvdbrug opened this issue Jul 25, 2024 · 4 comments · Fixed by #3200
Closed

Mollie Plugin doesn't check for eligibility #2973

martijnvdbrug opened this issue Jul 25, 2024 · 4 comments · Fixed by #3200
Assignees
Labels
P3: minor Non-critical, no workarounds exist type: bug 🐛 Something isn't working

Comments

@martijnvdbrug
Copy link
Collaborator

Is your feature request related to a problem? Please describe.
The Mollie plugin should check if the Mollie payment method is eligible for the given order, before creating a payment intent. Currently, Vendure will throw an Ineligible Payment Method error after payment, but funds have already been withdrawn from the clients account.

Describe the solution you'd like
The createPaymentIntent() in mollie.service.ts should check if the payment method is eligible, if not, throw an INELIGIBLE_PAYMENT_METHOD_ERROR

@martijnvdbrug martijnvdbrug added the P3: minor Non-critical, no workarounds exist label Jul 25, 2024
@martijnvdbrug martijnvdbrug self-assigned this Jul 25, 2024
@casperiv0
Copy link
Contributor

We've had this happen several times in production the last 2 weeks. The customer would apply a coupon code which would give them 70% discount, then they remove that coupon code and add another coupon code which makes their order free. We don't use Mollie for free orders, but the Mollie transaction did still continue.

Does the Mollie plugin support free orders, since we could switch that over already I think. That would avoid the bug I just mentioned, or not entirely? @martijnvdbrug

@martijnvdbrug
Copy link
Collaborator Author

@casperiv0 The latest version of the Mollie plugin contains a pretty serious bug, and I am still in contact with Mollie support about it.

The latest version of the Mollie plugin supports $0 orders. In case of a $0 order, it just transitions the order to Settled, without ever calling Mollie. You should not use the latest version of the mollie plugin untill the above mentioned bug is fixed though.

@casperiv0
Copy link
Contributor

I think that's the reason why my bug happens, I think they're somewhat correlated?

@martijnvdbrug
Copy link
Collaborator Author

@casperiv0 I've contacted you on Discord, because I'm not sure if this is related

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P3: minor Non-critical, no workarounds exist type: bug 🐛 Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants