Skip to content

Comments

Update kve to 2.9.0, invalidates if key is empty#584

Merged
openshift-bot merged 1 commit intoopenshift:masterfrom
benjaminapetersen:kve-bump-disallow-empty-keys
Sep 23, 2016
Merged

Update kve to 2.9.0, invalidates if key is empty#584
openshift-bot merged 1 commit intoopenshift:masterfrom
benjaminapetersen:kve-bump-disallow-empty-keys

Conversation

@benjaminapetersen
Copy link
Contributor

Fixes #1374154
https://bugzilla.redhat.com/show_bug.cgi?id=1374154

Per this PR: openshift/angular-key-value-editor#116

Entries w/o a key will be invalid, the user will be notified and any form that has ng-disable w/ a reference to form.$invalid will require the user to resolve the issue before submitting.

@jwforres @spadgett

@jwforres
Copy link
Member

[merge]

@openshift-bot
Copy link

Evaluated for origin web console merge up to 5cbcc2d

@openshift-bot
Copy link

openshift-bot commented Sep 23, 2016

Origin Web Console Merge Results: SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_origin_web_console/430/)

@openshift-bot openshift-bot merged commit 1167e76 into openshift:master Sep 23, 2016
@benjaminapetersen benjaminapetersen deleted the kve-bump-disallow-empty-keys branch September 23, 2016 18:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants