Bug #13424

Add Patternfly donut charts support

Added by Daniel Lobato Garcia over 1 year ago. Updated about 1 year ago.

Status:Closed
Priority:Normal
Assigned To:Gail Steiger
Category:Web Interface
Target version:Team Daniel - iteration 3
Difficulty: Bugzilla link:
Found in release: Pull request:https://github.com/theforeman/foreman/pull/3603, https://github.com/theforeman/foreman/pull/3931
Story points-
Velocity based estimate-
Release1.14.0Release relationshipAuto

Description

Our donut charts do not look like they should in Patternfly. https://www.patternfly.org/patterns/donut-chart/ uses c3, so we can do that too.
Eventually I want to remove flot_pies too, but I have to yet figure out how to auto refresh donut charts.
Adding this library also will allow us to change our charts in Host#show to the newer style. https://www.patternfly.org/widgets/#basic-charts


Related issues

Related to OpenSCAP - Bug #16517: Explicitly require chart.js for 1.14 compatibility New 09/12/2016
Related to Foreman Remote Execution - Bug #16516: Explicitly require chart.js for 1.14 compatibility New 09/12/2016
Related to Foreman - Bug #10201: Statistics page behavior when looking at an empty org loo... Resolved 04/17/2015
Related to Foreman - Bug #17569: React Chart Component Error Closed 12/04/2016

Associated revisions

Revision 659d49a1
Added by Gail Steiger about 1 year ago

Fixes #13424 - c3 patternfly react implementation

Revision 3f9c4075
Added by Dominic Cleal about 1 year ago

refs #13424 - add wait_for_ajax to statistics page load test

Revision 9ff20c7f
Added by Dominic Cleal about 1 year ago

refs #13424 - add c3, react npm dependencies

Revision c5788de1
Added by Dominic Cleal about 1 year ago

refs #13424 - update patternfly-sass

History

#1 Updated by The Foreman Bot over 1 year ago

  • Status changed from New to Ready For Testing
  • Assigned To set to Daniel Lobato Garcia
  • Pull request https://github.com/theforeman/foreman/pull/3128 added

#2 Updated by Daniel Lobato Garcia over 1 year ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

#3 Updated by Dominic Cleal over 1 year ago

  • Status changed from Closed to Ready For Testing

PR was accidentally merged to develop in revision f4b2a116, reverted in 743c84c8.

#4 Updated by The Foreman Bot over 1 year ago

  • Pull request https://github.com/theforeman/foreman/pull/3603 added

#5 Updated by Daniel Lobato Garcia about 1 year ago

  • Target version set to Team Daniel - iteration 1

#6 Updated by Daniel Lobato Garcia about 1 year ago

  • Assigned To changed from Daniel Lobato Garcia to Gail Steiger
  • Pull request deleted (https://github.com/theforeman/foreman/pull/3128)

#7 Updated by Daniel Lobato Garcia about 1 year ago

  • Target version changed from Team Daniel - iteration 1 to Team Daniel - iteration 2

#8 Updated by Marek Hulán about 1 year ago

  • Related to Bug #16517: Explicitly require chart.js for 1.14 compatibility added

#9 Updated by Marek Hulán about 1 year ago

  • Related to Bug #16516: Explicitly require chart.js for 1.14 compatibility added

#10 Updated by Daniel Lobato Garcia about 1 year ago

  • Target version changed from Team Daniel - iteration 2 to Team Daniel - iteration 3

#11 Updated by Gail Steiger about 1 year ago

  • Status changed from Ready For Testing to Closed

#12 Updated by Dominic Cleal about 1 year ago

  • Release set to 1.14.0

#13 Updated by The Foreman Bot about 1 year ago

  • Pull request https://github.com/theforeman/foreman/pull/3931 added

#14 Updated by Marek Hulán 11 months ago

  • Related to Bug #10201: Statistics page behavior when looking at an empty org looks broken added

#15 Updated by Dominic Cleal 11 months ago

  • Related to Bug #17569: React Chart Component Error added

Also available in: Atom PDF