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

Filter Recent VMs/Hosts charts by provider #144

Conversation

h-kataria
Copy link
Contributor

On Infrastructure Provider Dashboard view, Recent hosts and Recent VMs should be filtered by selected provider.

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

before:
dashboard_infra_before

after:
dashboard_infra_after

@dclarizio please review.

On Infrastructure Provider Dashboard view, Recent hosts and Recent VMs should be filtered by selected provider.

https://bugzilla.redhat.com/show_bug.cgi?id=1410251
@h-kataria h-kataria force-pushed the infra_dashboard_view_recent_chart_data_fix branch from eb53f79 to 2358831 Compare January 12, 2017 17:45
@miq-bot
Copy link
Member

miq-bot commented Jan 12, 2017

Checked commit h-kataria@2358831 with ruby 2.2.5, rubocop 0.37.2, and haml-lint 0.16.1
2 files checked, 0 offenses detected
Everything looks good. 🏆

@dclarizio dclarizio merged commit a6b13ac into ManageIQ:master Jan 12, 2017
@dclarizio dclarizio added this to the Sprint 52 Ending Jan 16, 2017 milestone Jan 12, 2017
@simaishi
Copy link
Contributor

Euwe backport (to manageiq repo) details:

$ git log -1
commit 93f0130d6e4f88201dca35ff29b042bbb600ad1a
Author: Dan Clarizio <[email protected]>
Date:   Thu Jan 12 14:43:37 2017 -0800

    Merge pull request #144 from h-kataria/infra_dashboard_view_recent_chart_data_fix
    
    Filter Recent VMs/Hosts charts by provider
    (cherry picked from commit a6b13acca42d1ff6ce506a8bb0e5efa18d9f47ac)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1413205

@h-kataria h-kataria deleted the infra_dashboard_view_recent_chart_data_fix branch February 7, 2017 17:58
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