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

Change <Other(x)> to Others in C&U pie chart labels #2171

Merged
merged 1 commit into from
Sep 14, 2017

Conversation

PanSpagetka
Copy link
Contributor

Change <Other(x)> to Others in C&U pie chart labels

Screenshots:

Before:
screenshot from 2017-09-13 11-35-01

After:
screenshot from 2017-09-13 11-42-39

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

@PanSpagetka PanSpagetka force-pushed the others-label-cu-pie-chart branch 2 times, most recently from 4a31d3a to e277eea Compare September 13, 2017 08:51
@miq-bot
Copy link
Member

miq-bot commented Sep 13, 2017

Checked commit PanSpagetka@e277eea with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
1 file checked, 0 offenses detected
Everything looks fine. 🍪

@mzazrivec mzazrivec self-assigned this Sep 14, 2017
@mzazrivec mzazrivec added this to the Sprint 69 Ending Sep 18, 2017 milestone Sep 14, 2017
@mzazrivec mzazrivec merged commit b7906e7 into ManageIQ:master Sep 14, 2017
simaishi pushed a commit that referenced this pull request Nov 13, 2017
Change <Other(x)> to Others in C&U pie chart labels
(cherry picked from commit b7906e7)

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

Fine backport details:

$ git log -1
commit b236a43b27ebad4126fca762f41134eaf42c9923
Author: Milan Zázrivec <[email protected]>
Date:   Thu Sep 14 10:27:49 2017 +0200

    Merge pull request #2171 from PanSpagetka/others-label-cu-pie-chart
    
    Change <Other(x)> to Others in C&U pie chart labels
    (cherry picked from commit b7906e7622ef9f2fa00060081a145f70fe8f65b8)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1496907

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