-
Notifications
You must be signed in to change notification settings - Fork 612
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
form validation fix #2475
form validation fix #2475
Conversation
@Dpschu2 please add a description to the PR and a corresponding screenshot/video confirming the fix. Thanks! |
Added the screenshot showing the field message that was missing. The description already describes what was added/missing. |
please also remove Also please do not forget to squash commits with one appropriate message |
@Dpschu2 do you mind doing a rebase? I tried to do it for you but I don't have access to your fork. This is good to go after rebase. |
What?
Adding name field validation message for product reviews form.
Screenshots