Skip to content

Bug 1605136 - Let users clear default fields with template service broker#711

Merged
openshift-merge-robot merged 1 commit intoopenshift:masterfrom
spadgett:schema-form-empty-values
Dec 2, 2018
Merged

Bug 1605136 - Let users clear default fields with template service broker#711
openshift-merge-robot merged 1 commit intoopenshift:masterfrom
spadgett:schema-form-empty-values

Conversation

@spadgett
Copy link
Copy Markdown
Member

@openshift-ci-robot openshift-ci-robot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Oct 27, 2018
@spadgett spadgett force-pushed the schema-form-empty-values branch from 450dc4d to 604921c Compare November 12, 2018 17:38
@openshift-ci-robot openshift-ci-robot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Nov 12, 2018
@spadgett
Copy link
Copy Markdown
Member Author

jenkins rebuild

@spadgett
Copy link
Copy Markdown
Member Author

The upstream origin change is in the merge queue. @rhamilto ptal

Copy link
Copy Markdown
Member

@rhamilto rhamilto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

autoFocus={autofocus}
value={value == null ? '' : value}
{...inputProps}
onChange={(event) => onChange(event.target.value)}
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nit: you don't use the parens around the argument below.

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Nov 28, 2018
@spadgett spadgett changed the title [WIP] Bug 1605136 - Let users clear default fields with template service broker Bug 1605136 - Let users clear default fields with template service broker Dec 2, 2018
@openshift-ci-robot openshift-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Dec 2, 2018
@openshift-merge-robot openshift-merge-robot merged commit 765952b into openshift:master Dec 2, 2018
@spadgett spadgett deleted the schema-form-empty-values branch December 2, 2018 18:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

4 participants