Skip to content

Conversation

@rhamilto
Copy link
Member

@rhamilto rhamilto commented Feb 5, 2019

@openshift-ci-robot openshift-ci-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Feb 5, 2019
@openshift-merge-robot
Copy link
Contributor

/retest

2 similar comments
@openshift-merge-robot
Copy link
Contributor

/retest

@rhamilto
Copy link
Member Author

rhamilto commented Feb 6, 2019

/retest

@rhamilto
Copy link
Member Author

rhamilto commented Feb 6, 2019

/retest

Copy link
Member

Choose a reason for hiding this comment

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

I don't think we need the call to super.componentWillUnmount() since this isn't extending SafetyFirst

Copy link
Member Author

Choose a reason for hiding this comment

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

Updated

Copy link
Member

@spadgett spadgett left a comment

Choose a reason for hiding this comment

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

/lgtm

👍

Copy link
Member

Choose a reason for hiding this comment

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

Thanks for fixing that

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Feb 7, 2019
@openshift-ci-robot openshift-ci-robot removed the lgtm Indicates that a PR is ready to be merged. label Feb 7, 2019
@spadgett
Copy link
Member

spadgett commented Feb 7, 2019

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Feb 7, 2019
@rhamilto
Copy link
Member Author

rhamilto commented Feb 7, 2019

Sorry. I'm a dolt and committed and force pushed in the wrong terminal tab, resulting in this change. There are no new changes.

@spadgett
Copy link
Member

spadgett commented Feb 7, 2019

/retest

2 similar comments
@spadgett
Copy link
Member

spadgett commented Feb 7, 2019

/retest

@spadgett
Copy link
Member

spadgett commented Feb 7, 2019

/retest

@spadgett
Copy link
Member

spadgett commented Feb 8, 2019

The local console couldn't find index.html on startup, hm.

index.html not found in configured public-dir path: open frontend/public/dist/index.html: no such file or directory

/retest

@openshift-merge-robot openshift-merge-robot merged commit 16a39af into openshift:master Feb 8, 2019
@rhamilto rhamilto deleted the console-1020 branch February 18, 2019 13:57
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/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants