Skip to content

Update utilization chart available value when used or total changes #144

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

Merged
merged 2 commits into from
Oct 28, 2015

Conversation

erundle
Copy link

@erundle erundle commented Oct 28, 2015

This has already been merged into master. Adding to 2.3.x so that we can create a patch release

@spadgett

jeff-phillips-18 added a commit that referenced this pull request Oct 28, 2015
Update utilization chart available value when used or total changes
@jeff-phillips-18 jeff-phillips-18 merged commit 2c9c63f into patternfly:branch-2.3.x Oct 28, 2015
@spadgett
Copy link
Contributor

Hi, @erundle. It looks like your commit is different from #143. The change needs to be in utilization-chart-directive.js.

@erundle
Copy link
Author

erundle commented Oct 28, 2015

@spadgett what is different about it?

@spadgett
Copy link
Contributor

The change is not in the correct file. It should be in utilization-chart-directive.js, not donut-pct-chart-directive.js.

@erundle
Copy link
Author

erundle commented Oct 28, 2015

@spadgett nevermind it looks like we have the bug in two places and show be applied to both files

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants