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

[Checkout] Impossible to checkout on shop 3570 #9610

Closed
audez opened this issue Aug 30, 2022 · 17 comments
Closed

[Checkout] Impossible to checkout on shop 3570 #9610

audez opened this issue Aug 30, 2022 · 17 comments
Labels
bug-s2 The bug is affecting any of the non-critical features described in S1 and there is no workaround.

Comments

@audez
Copy link
Collaborator

audez commented Aug 30, 2022

Description

Clients can't checkout on shop 3570
https://coopcircuits.fr/lafermemaraichette/shop#/shop
It doesn't work whether choosing to pay with bank card or with cash.

Expected Behavior

You should be able to checkout.

Actual Behaviour

When trying with voluntary contribution or bank card, same error:
"The checkout failed. Please let us know so we can proceed your order".

Request URL: https://coopcircuits.fr/checkout.json
{"errors":{},"flash":{"error":"The checkout failed. Please let us know so that we can process your order."}}

Then, after trying once with bank card and trying again, the error is different:
"Payment could not be processed, please check the details you entered"

Request URL: https://coopcircuits.fr/checkout.json
{"errors":{},"flash":{"error":"Payment could not be processed, please check the details you entered"}}

Steps to Reproduce

  1. Try to place an order in the shop https://coopcircuits.fr/lafermemaraichette/shop#/shop

Animated Gif/Screenshot

Screen Shot 2022-08-30 at 13 46 09
Screen Shot 2022-08-30 at 13 45 40

Severity

bug-s1: a critical feature is broken: checkout, payments, signup, login

Additional context

The shop manager indicated that the first issue was reported on Friday by one user, then yesterday at least 2 clients reported not being able to checkout. Before that she never had any issue. (can it be linked to the downtime on CoopCircuits ? #9596)

@audez audez added the bug-s1 The bug is stopping the platform from working, and there is no workaround. Impact of lot of users. label Aug 30, 2022
@audez audez changed the title [Checkout] Impossible to place an order on shop 3570 [Checkout] Impossible to checkout on shop 3570 Aug 30, 2022
@filipefurtad0
Copy link
Contributor

filipefurtad0 commented Aug 30, 2022

The second error Payment could not be processed, please check the details you entered looks like this on bugsnag:
https://app.bugsnag.com/open-food-france/coopcircuits/errors/62ecc2b4c7b65f000c45d1d2?filters[event.since]=30d&filters[error.status]=open&event_id=630dfa3600963db5be290000

@audez
Copy link
Collaborator Author

audez commented Aug 30, 2022

thanks! yes indeed, i see me in the request details :) do you know where it can come from?
It corresponds to the second error ("Payment could not be processed, please check the details you entered")
Weird thing is that i don't see the bug with the first error ("The checkout failed. Please let us know so that we can process your order.")

@audez
Copy link
Collaborator Author

audez commented Aug 30, 2022

oh oh !

Screen Shot 2022-08-30 at 14 08 56

@audez
Copy link
Collaborator Author

audez commented Aug 30, 2022

ok I get it: if you choose the option "back office only" in the shipping methods, it displays the previous error in the shipping info, and then you can't checkout
Edit: i don't actually, i still can't checkout 😭

Screen Shot 2022-08-30 at 14 13 14

@filipefurtad0
Copy link
Contributor

Whoa - awesome and quick investigation @audez ⚡ 🙌

This are some related S3s on this, it appeared before... I'll look for them.

@RachL
Copy link
Contributor

RachL commented Aug 30, 2022

well done @audez then I guess we can close here? This switch is intended for people to be able to disable their shipping methods from time to time. There are already issues covering the fact that we need to improve the error messages 👍

@audez
Copy link
Collaborator Author

audez commented Aug 30, 2022

no actually it's a fail 😂 i still can't checkout, even with the other option, or after creating new shipping method and payment method

@RachL
Copy link
Contributor

RachL commented Aug 30, 2022

@audez I can checkout fine. Can you detail the step you do to reproduce?

@RachL RachL added bug-s2 The bug is affecting any of the non-critical features described in S1 and there is no workaround. and removed bug-s1 The bug is stopping the platform from working, and there is no workaround. Impact of lot of users. labels Aug 30, 2022
@audez
Copy link
Collaborator Author

audez commented Aug 30, 2022

oh ok cool, in fact it works for me too. It's because i kept the same cart than when i first had the issue. I was able to reproduce only with the same cart with which I had the bug. When i added something to the cart, it was OK!

@filipefurtad0
Copy link
Contributor

filipefurtad0 commented Aug 30, 2022

Should we double check if those customers were not charged with Stripe, after the first failed checkout attempt? We had something similar here. We did have a recent Stripe upgrade...

@filipefurtad0
Copy link
Contributor

I think this might be the same or very closely related to this issue:
#8219

@RachL
Copy link
Contributor

RachL commented Aug 30, 2022

We did have a recent Stripe upgrade...

I don't think it was deployed on FR?

@filipefurtad0
Copy link
Contributor

I don't think it was deployed on FR?

Right! - it didn't - we can scratch that 👍

@audez
Copy link
Collaborator Author

audez commented Aug 30, 2022

oh yes it looks exactly the same than 8219 ! it's an old one then :) thanks a lot. Just for some context, the shop manager said she disabled displayin shipping methods on Friday, because she has 2 OC, 2 delivery days and 2 corresponding shipping methods, but sometimes people don't choose the good one corresponding to the OC..And as the delivery date doesn't appear in the reports, she was a bit stuck. This should be sorted out with possibility of linking an OC to a shipping method #8974. For now she will create only one shipping method.

@audez
Copy link
Collaborator Author

audez commented Aug 30, 2022

You're right @filipefurtad0 I just checked and there has been Stripe payments 😢 and i'm a victim too!

@RachL
Copy link
Contributor

RachL commented Aug 30, 2022

shall we update 8219 or do we want to open a new issue that merges the two?

@filipefurtad0
Copy link
Contributor

Ok, tried to sum up everything here:
#9614

Closing here and 8219.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug-s2 The bug is affecting any of the non-critical features described in S1 and there is no workaround.
Projects
None yet
Development

No branches or pull requests

3 participants