Skip to content

Disable service-instance delete link when already schedualed for deletion#2234

Merged
openshift-merge-robot merged 1 commit intoopenshift:masterfrom
jhadvig:service-instance
Oct 16, 2017
Merged

Disable service-instance delete link when already schedualed for deletion#2234
openshift-merge-robot merged 1 commit intoopenshift:masterfrom
jhadvig:service-instance

Conversation

@jhadvig
Copy link
Member

@jhadvig jhadvig commented Oct 9, 2017

@spadgett just FYI ... will properly test once the web-catalog is fixed

Closes #2227

@openshift-ci-robot openshift-ci-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Oct 9, 2017
@jhadvig jhadvig requested a review from spadgett October 9, 2017 15:08
@spadgett
Copy link
Member

spadgett commented Oct 9, 2017

LGTM

Copy link
Contributor

@cdcabrera cdcabrera left a comment

Choose a reason for hiding this comment

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

sweeet, looking good (code wise)!

@jhadvig
Copy link
Member Author

jhadvig commented Oct 10, 2017

.... adding dist

@openshift-bot
Copy link

Origin Web Console Action Required: Pull request cannot be automatically merged, please rebase your branch from latest HEAD and push again

@openshift-bot openshift-bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Oct 12, 2017
@spadgett
Copy link
Member

Hey @jhadvig I wasn't sure if you had been able to verify the fix or if we're still waiting? Let me know.

It does need rebase now.

@spadgett spadgett changed the title Disable service-instance delete link when already schedulaed for deletion Disable service-instance delete link when already schedualed for deletion Oct 13, 2017
@openshift-merge-robot openshift-merge-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Oct 16, 2017
@jhadvig
Copy link
Member Author

jhadvig commented Oct 16, 2017

@jeff-phillips-18 @spadgett I've tested the change and it works as expected. if the service-instance is deleted then the Delete link is disabled.

Screen:
disable

@spadgett
Copy link
Member

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Oct 16, 2017
@spadgett
Copy link
Member

Flake #1685

/retest

@openshift-merge-robot
Copy link
Contributor

Automatic merge from submit-queue.

@openshift-merge-robot openshift-merge-robot merged commit ffac257 into openshift:master Oct 16, 2017
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/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Should not be able to delete an already deleted service

7 participants