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

Images for pluggable provider domains in Automate Explorer #11618

Merged
merged 1 commit into from Oct 3, 2016

Conversation

mkanoor
Copy link
Contributor

@mkanoor mkanoor commented Sep 30, 2016

Based on the top_level_namespace, use the appropriate vendor icon
Added a new manageiq png

https://www.pivotaltracker.com/n/projects/1613499/stories/131438349

Before
old_view

After
new_view

Based on PR:
#11368

Based on the top_level_namespace, use the appropriate vendor icon
Added a new manageiq png
@mkanoor
Copy link
Contributor Author

mkanoor commented Sep 30, 2016

@h-kataria @epwinchell @tinaafitz
Please review

@mkanoor
Copy link
Contributor Author

mkanoor commented Sep 30, 2016

@gmcculloug

@miq-bot
Copy link
Member

miq-bot commented Sep 30, 2016

Checked commit mkanoor@b790b36 with ruby 2.2.5, rubocop 0.37.2, and haml-lint 0.16.1
2 files checked, 0 offenses detected
Everything looks good. 🍪

@tinaafitz
Copy link
Member

@mkanoor Looks great!

@epwinchell
Copy link
Contributor

@mkanoor Looks good

@epwinchell
Copy link
Contributor

@miq-bot unassign @epwinchell

@miq-bot
Copy link
Member

miq-bot commented Sep 30, 2016

@epwinchell unrecognized command 'unassign', ignoring...

Accepted commands are: add_label, remove_label, rm_label, assign, set_milestone

@h-kataria
Copy link
Contributor

looks good

@h-kataria h-kataria added this to the Sprint 47 Ending Oct 3, 2016 milestone Oct 3, 2016
@h-kataria h-kataria merged commit 6868d8f into ManageIQ:master Oct 3, 2016
@chessbyte
Copy link
Member

@mkanoor @h-kataria please label euwe/yes or euwe/no

chessbyte pushed a commit that referenced this pull request Oct 5, 2016
Images for pluggable provider domains in Automate Explorer
(cherry picked from commit 6868d8f)
@chessbyte
Copy link
Member

Euwe Backport details:

$ git log
commit c98ea7bf6cdf6cd0de2a4b99577d6eeb9728330b
Author: Harpreet Kataria <hkataria@redhat.com>
Date:   Mon Oct 3 18:27:06 2016 -0400

    Merge pull request #11618 from mkanoor/domain_icons

    Images for pluggable provider domains in Automate Explorer
    (cherry picked from commit 6868d8f6e91d144066f535c6b8773400e05fe2fa)

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

6 participants