-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
Change validation timing on create from image page #4968
Conversation
links. Show the required error if the user enters and deletes the value, even if the field still | ||
has focus. | ||
--> | ||
<div ng-class="{'has-error': (form.appname.$error.required && form.appname.$dirty) || (form.appname.$invalid && validateName) || nameTaken}"> |
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.
the meaning of validateName confused me at first because it reads as an action. maybe shouldValidateName ?
Avoid problems with the try it and more options links by changing when we validate fields.
680f230
to
e574657
Compare
@jwforres updated from your comments |
[merge] |
continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_requests_origin/3512/) (Image: devenv-fedora_2451) |
Evaluated for origin merge up to e574657 |
[Test]ing while waiting on the merge queue |
continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_origin/5445/) |
Evaluated for origin test up to e574657 |
Avoid problems with the try it and more options links by changing when we validate fields.
Fixes #4912