Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[EUWE] Remove disabling of 'instance_retire' button #14016

Merged
merged 1 commit into from
Feb 22, 2017

Conversation

romanblanco
Copy link
Member

Leaves the button "Set Retirement Date" enabled even for a retired instance

Links

https://bugzilla.redhat.com/show_bug.cgi?id=1306471

Steps for Testing/QA

  1. retire an instance
  2. go to that instance details page
  3. click Lifecycle
  4. button Set Retirement Date should be enabled

(see BZ ticket for more info)

@romanblanco romanblanco changed the title Remove disabling of 'instance_retire' button to [EUWE] Remove disabling of 'instance_retire' button [EUWE] Remove disabling of 'instance_retire' button Feb 22, 2017
@romanblanco
Copy link
Member Author

@miq-bot add_label bug, ui
@miq-bot assign @simaishi

@PanSpagetka or @vecerek can you review the fix, please?

@miq-bot
Copy link
Member

miq-bot commented Feb 22, 2017

Checked commit romanblanco@7610fda with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
1 file checked, 0 offenses detected
Everything looks good. ⭐

@PanSpagetka
Copy link
Contributor

PanSpagetka commented Feb 22, 2017

Verified in UI, it works 👍

@martinpovolny martinpovolny added this to the Sprint 55 Ending Feb 27, 2017 milestone Feb 22, 2017
@martinpovolny martinpovolny merged commit d40f9bb into ManageIQ:euwe Feb 22, 2017
@romanblanco romanblanco deleted the bz1306471 branch February 22, 2017 15:09
@simaishi
Copy link
Contributor

@romanblanco is there a 'master' PR for this?

@romanblanco
Copy link
Member Author

@simaishi No, it's not. Just this one, in master this code is already removed

@simaishi
Copy link
Contributor

@romanblanco would you mind referencing the PR that removed this from 'master'?

@romanblanco
Copy link
Member Author

@simaishi Sure, thanks 👍

From master removed by in commit d513a3b

@romanblanco
Copy link
Member Author

@simaishi Actually, I've just noticed that the issue has been moved to master at the PR: https://github.com/ManageIQ/manageiq/pull/13076/files#diff-90329e2b42ab1320a3c025ef617c5e88R5

I'll create PR for master too

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants