Skip to content

Conversation

spadgett
Copy link
Member

@spadgett spadgett commented Nov 1, 2017

Fixes a problem where the regex used for validating label names is
wrong. Names with a / were incorrectly blocked.

See discussion in #2414

openshift web console 2017-11-01 12-12-24

openshift web console 2017-11-01 12-19-48

/kind bug
/assign @jwforres
cc @mjudeikis

@openshift-ci-robot openshift-ci-robot added kind/bug Categorizes issue or PR as related to a bug. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Nov 1, 2017
@jwforres
Copy link
Member

jwforres commented Nov 1, 2017

The Learn More feels out of place as a block, try the inline one instead.

@spadgett
Copy link
Member Author

spadgett commented Nov 1, 2017

@jwforres Updated to use a checkbox instead:

openshift web console 2017-11-01 12-45-38

Fixes a problem where the regex used for validating label names is
wrong. Names with a `/` were incorrectly blocked.
@jwforres
Copy link
Member

jwforres commented Nov 1, 2017

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Nov 1, 2017
@mjudeikis
Copy link

@rusenask :) FIY . Done.

@openshift-merge-robot
Copy link
Contributor

Automatic merge from submit-queue.

@openshift-merge-robot openshift-merge-robot merged commit b5efba1 into openshift:master Nov 1, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug. lgtm Indicates that a PR is ready to be merged. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants