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

BZ#1653288 Service is getting retired without any confirmation in self service ui #1502

Conversation

AllenBW
Copy link
Member

@AllenBW AllenBW commented Nov 27, 2018

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

A confirmation modal was never present on this command (the retire now menu item). But the response to the action, the hardcoded text was inaccurate, the service isn't retired upon clicking, a retire service request is created, update the response notifications to use the api feedback rather than hardcoded text.

Message on success

screen shot 2018-11-27 at 3 09 19 pm

Message on error

screen shot 2018-11-27 at 3 17 33 pm

@AllenBW AllenBW requested a review from himdel November 27, 2018 20:20
@AllenBW AllenBW added this to the Sprint 100 Ending Dec 3, 2018 milestone Nov 27, 2018
@miq-bot
Copy link
Member

miq-bot commented Nov 27, 2018

Checked commit AllenBW@da7fc9c with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
0 files checked, 0 offenses detected
Everything looks fine. ⭐

Copy link
Contributor

@himdel himdel left a comment

Choose a reason for hiding this comment

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

LGTM, matches what I'm seeing :)

@himdel himdel merged commit d163312 into ManageIQ:master Nov 28, 2018
@AllenBW AllenBW deleted the BZ/#1653288-Serviceisgettingretiredwithoutanyconfirmationinselfserviceui branch November 28, 2018 15:52
simaishi pushed a commit that referenced this pull request Dec 3, 2018
…iredwithoutanyconfirmationinselfserviceui

BZ#1653288 Service is getting retired without any confirmation in self service ui

(cherry picked from commit d163312)

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1653288
@simaishi
Copy link
Contributor

simaishi commented Dec 3, 2018

Hammer backport details:

$ git log -1
commit 5efed98f06eaa61f7bf82f96c590e11c0ca34920
Author: Martin Hradil <[email protected]>
Date:   Wed Nov 28 16:43:53 2018 +0100

    Merge pull request #1502 from AllenBW/BZ/#1653288-Serviceisgettingretiredwithoutanyconfirmationinselfserviceui
    
    BZ#1653288 Service is getting retired without any confirmation in self service ui
    
    (cherry picked from commit d163312851023d71fef6781da1b408656b0d9252)
    
    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1653288

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.

4 participants