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

Added "Escalate Privilege" checkbox #1207

Merged
merged 1 commit into from
May 1, 2017

Conversation

h-kataria
Copy link
Contributor

@h-kataria h-kataria commented May 1, 2017

Added "Escalate Privilege" checkbox switch on both Provisioning & Retirement tabs in form and on Summary screen to display the selected values. Default value for the new checkbox is false.

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

@gmcculloug @bzwei please review/test.

backend work in ManageIQ/manageiq#14929

screenshot:
catalog_item_form_prov_tab

catalog_item_form_ret_tab

catalog_item_summary_prov_tab

catalog_item_summary_retirement_tab

@bzwei
Copy link
Contributor

bzwei commented May 1, 2017

@h-kataria For retirement Escalate Privilege option should be hidden or disabled if no playbook is selected.

@h-kataria
Copy link
Contributor Author

@bzwei please re-review

@h-kataria h-kataria force-pushed the escalate_privilege_option branch 2 times, most recently from 9aabe1a to 6acd19a Compare May 1, 2017 14:58
Added "Escalate Privilege" checkbox switch on both Provisioning & Retirement tabs in form and on Summary screen to display the selected values. Default value for the new checkbox is false.

https://bugzilla.redhat.com/show_bug.cgi?id=1445377
@miq-bot
Copy link
Member

miq-bot commented May 1, 2017

Checked commit h-kataria@7523a3a with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
4 files checked, 0 offenses detected
Everything looks fine. 👍

@bzwei
Copy link
Contributor

bzwei commented May 1, 2017

Tested and works!

@dclarizio dclarizio merged commit 4f45c53 into ManageIQ:master May 1, 2017
@dclarizio dclarizio added this to the Sprint 60 Ending May 8, 2017 milestone May 1, 2017
simaishi pushed a commit that referenced this pull request May 1, 2017
@simaishi
Copy link
Contributor

simaishi commented May 1, 2017

Fine backport details:

$ git log -1
commit 3e989229cdf308b93118362d41971f77fdf363b2
Author: Dan Clarizio <dclarizi@redhat.com>
Date:   Mon May 1 09:19:39 2017 -0700

    Merge pull request #1207 from h-kataria/escalate_privilege_option
    
    Added "Escalate Privilege" checkbox
    (cherry picked from commit 4f45c537341088d8f1dfba252f4ae77822998bd8)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1447086

@h-kataria h-kataria deleted the escalate_privilege_option branch May 3, 2017 15:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants