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

Fix name of 'Physical Infrastructure Providers' on the Physical Infrastructure Providers page #3574

Merged
merged 1 commit into from
Mar 12, 2018

Conversation

douglasgabriel
Copy link
Member

Bug Fix

This PR fixes the error described here https://bugzilla.redhat.com/show_bug.cgi?id=1525116

@douglasgabriel
Copy link
Member Author

@miq-bot add_label bug, gaprindashvili/yes

@douglasgabriel
Copy link
Member Author

@miq-bot assign @himdel

N_('Remove selected Infrastructure Providers from Inventory'),
N_('Remove Infrastructure Providers from Inventory'),
N_('Remove selected Physical Infrastructure Providers from Inventory'),
N_('Remove Physical Infrastructure Providers from Inventory'),
:url_parms => "main_div",
:send_checked => true,
:confirm => N_("Warning: The selected Infrastructure Providers and ALL of their components will be permanently removed!"),
Copy link
Contributor

@himdel himdel Mar 12, 2018

Choose a reason for hiding this comment

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

Also here? ;) And lines 12 and 16...

Copy link
Member Author

Choose a reason for hiding this comment

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

Thank you @himdel , I also add in these lines

@douglasgabriel douglasgabriel force-pushed the fix_physical_infra_name branch from 0c8668d to 16d4ddf Compare March 12, 2018 14:28
@miq-bot
Copy link
Member

miq-bot commented Mar 12, 2018

Checked commit douglasgabriel@16d4ddf with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
1 file checked, 0 offenses detected
Everything looks fine. 🏆

@himdel
Copy link
Contributor

himdel commented Mar 12, 2018

LGTM, waiting for green travis, thanks :)

@himdel himdel merged commit dc91c0e into ManageIQ:master Mar 12, 2018
@himdel himdel added this to the Sprint 81 Ending Mar 12, 2018 milestone Mar 12, 2018
simaishi pushed a commit that referenced this pull request Mar 12, 2018
Fix name of 'Physical Infrastructure Providers' on the Physical Infrastructure Providers page
(cherry picked from commit dc91c0e)

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

Gaprindashvili backport details:

$ git log -1
commit 6b38d4c2d5c25428bff1a1621ab212d69fd90b99
Author: Martin Hradil <[email protected]>
Date:   Mon Mar 12 15:26:46 2018 +0000

    Merge pull request #3574 from douglasgabriel/fix_physical_infra_name
    
    Fix name of 'Physical Infrastructure Providers' on the Physical Infrastructure Providers page
    (cherry picked from commit dc91c0e1757192477ce81bc2e7495c9e03fa9863)
    
    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1554454

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