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

GTL: add options needed for quadicon rendering. #2669

Merged
merged 1 commit into from
Nov 9, 2017

Conversation

martinpovolny
Copy link
Member

To properly render quadicons a set of controller instance variables is needed :-(

This PR adds this to the /report_data interface.

This is a partial fix for:

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

There are more problems with that area.

@ZitaNemeckova is working on one issue and @karelhala is working on another.

@miq-bot
Copy link
Member

miq-bot commented Nov 9, 2017

Checked commit martinpovolny@3a4cbd1 with ruby 2.3.3, rubocop 0.47.1, and haml-lint 0.20.0
3 files checked, 0 offenses detected
Everything looks fine. 🍰

@mzazrivec mzazrivec self-assigned this Nov 9, 2017
@mzazrivec mzazrivec added this to the Sprint 73 Ending Nov 13, 2017 milestone Nov 9, 2017
@mzazrivec mzazrivec merged commit 581d682 into ManageIQ:master Nov 9, 2017
simaishi pushed a commit that referenced this pull request Nov 15, 2017
@simaishi
Copy link
Contributor

Gaprindashvili backport details:

$ git log -1
commit 11ba887b1f6a7ff324dfd7a3f612d6113c843c85
Author: Milan Zázrivec <mzazrivec@redhat.com>
Date:   Thu Nov 9 10:24:30 2017 +0100

    Merge pull request #2669 from martinpovolny/quadicon_options
    
    GTL: add options needed for quadicon rendering.
    (cherry picked from commit 581d682147e7db2bfd146cc63a1e3c9b2c1f322c)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1513590
    https://bugzilla.redhat.com/show_bug.cgi?id=1513591

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