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

Show name in success catalog form messages #6352

Merged
merged 1 commit into from Nov 5, 2019

Conversation

rvsia
Copy link
Contributor

@rvsia rvsia commented Oct 30, 2019

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

Steps to Reproduce:
1.Go to Services -> Catalogs -> Catalogs accordion
2.Add or Update a catalog
3.View the success message

Actual results:
Catalog was saved

Expected results:
Catalog "{catalog_name}" was saved

Before

image

After

image

@miq-bot add_label bug, ivanchuk/yes, changelog/yes

@h-kataria

@miq-bot
Copy link
Member

miq-bot commented Oct 30, 2019

Checked commit rvsia@d111a71 with ruby 2.4.6, rubocop 0.69.0, haml-lint 0.20.0, and yamllint 1.10.0
1 file checked, 0 offenses detected
Everything looks fine. 👍

@rvsia
Copy link
Contributor Author

rvsia commented Oct 30, 2019

The CI failure is in master too, not caused by this PR. (Should be fixed by #6353 )

@rvsia rvsia closed this Oct 30, 2019
@rvsia rvsia reopened this Oct 30, 2019
@rvsia rvsia closed this Oct 31, 2019
@rvsia rvsia reopened this Oct 31, 2019
@rvsia
Copy link
Contributor Author

rvsia commented Nov 5, 2019

@ZitaNemeckova

@rvsia rvsia closed this Nov 5, 2019
@rvsia rvsia reopened this Nov 5, 2019
Copy link
Contributor

@ZitaNemeckova ZitaNemeckova left a comment

Choose a reason for hiding this comment

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

Works in UI 👍 Code looks good 👍

@himdel himdel self-assigned this Nov 5, 2019
@himdel himdel added this to the Sprint 124 Ending Nov 11, 2019 milestone Nov 5, 2019
@himdel himdel merged commit aa9be06 into ManageIQ:master Nov 5, 2019
@rvsia rvsia deleted the 17666276catalog-name branch December 10, 2019 14:24
simaishi pushed a commit that referenced this pull request Dec 16, 2019
Show name in success catalog form messages

(cherry picked from commit aa9be06)

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

Ivanchuk backport details:

$ git log -1
commit c1fc0f30d091651e5e773076ebd5783926f342e8
Author: Martin Hradil <mhradil@redhat.com>
Date:   Tue Nov 5 14:39:04 2019 +0100

    Merge pull request #6352 from rvsia/17666276catalog-name

    Show name in success catalog form messages

    (cherry picked from commit aa9be0607736cdc2d6bacbe6619fa8558992f289)

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

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.

None yet

5 participants