-
Notifications
You must be signed in to change notification settings - Fork 9.4k
Issue #5144 - fix add to cart product and validation required attribute for version CE 2.1 #10917
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
Conversation
@@ -28,7 +28,7 @@ | |||
</div> | |||
</div> | |||
<?php endif; ?> | |||
<div class="actions"> | |||
<div class="actions siedem"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
siedem
😉
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some debugging stuff is still there.
Also we have internal issue MAGETWO-70892 that currently resolved and in process of delivery. |
@omiroshnichenko I've tried to implement solution from the commit you had mentioned but it didn't work in 2.1. I think it requires more changes. So if we already have 2.2 released, we can close this issue |
My mistake. In updated 2.1-develop adding to cart using ajax and validation working fine. No more issue here |
Fix add to cart product and validation required attribute for version CE 2.1
Description
While ajax add to cart on product page is enable, the validation of required arguments not working properly - is doubled, first it shows errors message in empty inputs, then reload the page.
In version 2.2. this problem doesn't appear.
Fixed Issues (if relevant)
#5144
Manual testing scenarios