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

Add MiqWdigetSet to Dashboard dictionary translation #16784

Merged
merged 1 commit into from
Jan 10, 2018

Conversation

GregP
Copy link
Contributor

@GregP GregP commented Jan 9, 2018

Fixes and adds missing dictionary definition for Dashboard (MiqWidgetSet) for Delete action in UI.

This PR is for Delete Dashboard action code fix, please see related PR for Add/Edit actions here: ManageIQ/manageiq-ui-classic#2688

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

Screen shot prior to code fix:
dashboard dictrionary entry missing display

Screen shot post code fix:
dashboard dictionary entry display post code fix

@h-kataria
Copy link
Contributor

looks good, can be merged after Travis turns green

@miq-bot
Copy link
Member

miq-bot commented Jan 9, 2018

Checked commit GregP@538e65d 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. 👍

@h-kataria h-kataria self-assigned this Jan 10, 2018
@h-kataria h-kataria added this to the Sprint 77 Ending Jan 15, 2018 milestone Jan 10, 2018
@h-kataria h-kataria merged commit cab9cf9 into ManageIQ:master Jan 10, 2018
simaishi pushed a commit that referenced this pull request Jan 10, 2018
Add MiqWdigetSet to Dashboard dictionary translation
(cherry picked from commit cab9cf9)

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

Gaprindashvili backport details:

$ git log -1
commit 0fc4b7b87a911f1eca44bdb7ac888064d03517fd
Author: Harpreet Kataria <hkataria@redhat.com>
Date:   Wed Jan 10 10:55:49 2018 -0500

    Merge pull request #16784 from GregP/add_dashboard_dictionary_entry
    
    Add MiqWdigetSet to Dashboard dictionary translation
    (cherry picked from commit cab9cf94a03180668a9882be1c77acde3a69fab3)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1513603

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