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

Fixes page header when displaying Cloud Networks list #4256

Merged
merged 1 commit into from
Jul 6, 2018

Conversation

GregP
Copy link
Contributor

@GregP GregP commented Jul 5, 2018

Fixes incorrect action value passed when building page header text for Cloud Networks list view.

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

Screen shot prior to code fix:
cloud networks incorrect page header prior to code fix

Screen shot post code fix:
cloud networks correct page header post code fix

@miq-bot
Copy link
Member

miq-bot commented Jul 5, 2018

Checked commit GregP@f55b1c5 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. 🏆

@JPrause
Copy link
Member

JPrause commented Jul 6, 2018

@miq-bot add_label blocker

@JPrause
Copy link
Member

JPrause commented Jul 6, 2018

@GregP if this can be backported, can you add the gaprindashvili/yes label.

@miq-bot miq-bot added the blocker label Jul 6, 2018
@GregP
Copy link
Contributor Author

GregP commented Jul 6, 2018

@h-kataria Ready for review.

@h-kataria h-kataria self-assigned this Jul 6, 2018
@h-kataria
Copy link
Contributor

looks good

@h-kataria h-kataria added this to the Sprint 90 Ending Jul 16, 2018 milestone Jul 6, 2018
@h-kataria h-kataria merged commit 5fa5cad into ManageIQ:master Jul 6, 2018
simaishi pushed a commit that referenced this pull request Jul 11, 2018
Fixes page header when displaying Cloud Networks list
(cherry picked from commit 5fa5cad)

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

simaishi commented Jul 11, 2018

Gaprindashvili backport details:

$ git log -1
commit 17bb0d46775ea4b0daf72ee054f742f5a7b362bf
Author: Harpreet Kataria <hkataria@redhat.com>
Date:   Fri Jul 6 07:51:49 2018 -1000

    Merge pull request #4256 from GregP/cloud_networks_list_page_header
    
    Fixes page header when displaying Cloud Networks list
    (cherry picked from commit 5fa5cadbbb1c5a9870e2dda185cd9fc13cb8b37e)
    
    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1599353

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.

5 participants