-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Fix validation on PDP #2274
Comments
This should be part of the style guide, but for now we need a way to consistent display errors that require action. |
I should be able to enter $0.00 as a valid price on a product. Sometimes things are free. Right now I get a "Price is required" toast. |
We don't need to wait for design here, I think we all know the basics of how validation should work. The field should be focused, the error should show next to the field, etc. |
@zenweasel totally agree. If you guys take a stab at it and send it my way when it's done. But yes it should just as you describe. |
I just talked to @kieckhafer and he is willing to take this on. Hopefully we can expand the Autoform replacement work that @mikemurray has been working on |
@kieckhafer let me know when you work on this and we can talk through how the validation should be styled. I'm hoping this is something that does some automatic validation magic with our forms so we're consistent everywhere. |
Removed "critical impact" with the merge of #2531 |
This PR (#2893) should be able to close this issue once it's merged |
Also assigned @kieckhafer since he's fixing this in his PR #2893 |
Closed by #2893 |
All fields that are required or have a set type need to provide some user feedback.
The text was updated successfully, but these errors were encountered: