-
-
Notifications
You must be signed in to change notification settings - Fork 730
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
[Split Checkout] Error 422 on step '1 - Your details' - Update highlighted fields #9056
Comments
Can't reproduce this one on my local setup. If you could reproduce it on the staging server, please could you:
Thanks! 🙏 |
EDIT: Yes, I am sure I filled all fields. Here is a screen video: This is the console (after seeing the error twice): This is the cURL of the 422-POST entry (sorry for the formatting - can I improve this?): |
|
Oh, I do see it in Chromium as well, when I log out in Firefox an log in to the same user in Chromium and try to check out there. Here are some logs:
|
I've tried checking out on that shop with another user and experienced no issues. No answers, but some thoughts below. I've looked on that order in the DB:
and to that user / customer:
What's strange here is billing and shipping address are empty, although this user has placed orders before. The order is stuck on the cart state, also no shipping or billing address. Since we're dealing with a pick-up delivery method the hub's address should become the ship address for that order. Don't know why this is happening. My only clue would be that
|
Ok, I couldn't reproduce this issue at all... |
I have this on staging UK now as well...
The thing is I repeated the steps above for kunde3 (without 'ß') and kunde4 (with 'ß') and it worked for them. So no big news on the root cause, but it is still there. |
Hi! I stumbled upon the same issue. In the first place, in shop 1, i couldn't access step 2 (payment method) because of the 422 error. FYI the shipping method I selected was a pickup method where the fields "Save as default shipping address" and "Shipping address same as billing address?" are not displayed. When i compared the requests sent for shop 1, some fields was not sent the first time: It's quite weird because the second time a "ship_address_same_as_billing" was sent whereas it's a pickup method and i never checked the "Shipping address same as billing address?" field which is not displayed. Maybe it's normal though to send it by default and that's the fact that it was missing in the first place that is the problem? Something i find weird too is that in the request body, when you check "Save as default billing address", there are two fields "order[save_bill_address]" that are sent: one with value 0, the other one with value 1. Same for "Shipping address same as billing address?" |
ok guys this completes and kinda confirm my previous comment. As you can see in the screenshots, two fields were missing from the 422 request: As @filipefurtad0 said in a previous comment that "Since we're dealing with a pick-up delivery method the hub's address should become the ship address for that order.", I guess that these fields are required to make the request work. And for a weird reason they're sometimes not sent..But if you trigger a request with a delivery shipping method, then you can make the pickup request work again. |
This seems related: It might be the same issue (422), as the error is only triggered when clicking There is an additional message, which is visible before attempting to proceed to the Clicking a second time proceeds to the |
So we know how to reproduce? I've tried with a saved card but couldn't... is there something else to set up? |
I was able to reproduce it on master for different cards/customer/hubs/browsers. It seems to be introduced by the recent design changes, at least I could not reproduce it before #10154. Moving on to a separate issue, as the present one exists before these changes. |
I couldn't reproduce this either. But I opened a pull request to improve the error message. So hopefully will know more if this happens again. Until then we can close this, I think. |
Description
While trying to check out with split checkout active on staging AU I have observed an Error 422 in the console, going along with the flash message: 'Saving failed, please update the highlighted fields.' It gets triggered when clicking 'Next - Payment method' but there is no highlighted field...
I have tried editing all visible fields and changing their content without success - I wasn't able to proceed to step '2 - payment method'.
This is not always seen, the steps to reproduce are currently unknown.
Expected Behavior
There shouldn't be an Error 422.
There shouldn't be a flash error asking to update highlighted fields when there are no highlighted fields..
Actual Behaviour
Error 422 is seen in the console and the flash error is displayed without any highlighted field.
Steps to Reproduce
Animated Gif/Screenshot
Workaround
No workaround.
Severity
Not sure, since it is not yet in production.
Your Environment
Possible Fix
The text was updated successfully, but these errors were encountered: