Project

General

Profile

Bug #1321

Have pie chart graphs link to the hosts the slices represent part 2

Added by Jacob McCann over 7 years ago. Updated over 7 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
-
Target version:
Difficulty:
Triaged:
No
Bugzilla link:
Pull request:
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

Feature 1279 was opened to have charts link to hosts they represent. This was implemented but not completely. Only charts on dashboard link to anything.

The idea was to have fact charts and charts in the statistics page link to the hosts they represent as well.


Related issues

Related to Foreman - Feature #1278: Have pie chart graphs link to the hosts the slices representClosed2011-10-31

Associated revisions

Revision 001923dd (diff)
Added by Amos Benari over 7 years ago

fixes #1321 add click event to the facts and stats pie chart. The click event redirect to the relevant hosts search that match the clicked slice.

Signed-off-by: Amos Benari <>

Revision a7f7ee02 (diff)
Added by Amos Benari over 7 years ago

fixes #1321 add click event to the facts and stats pie chart. The click event redirect to the relevant hosts search that match the clicked slice.

Signed-off-by: Amos Benari <>

History

#1 Updated by Jacob McCann over 7 years ago

Whoops, actually meant this is in reference to feature 1278 ... which I have properly linked.

#2 Updated by Jacob McCann over 7 years ago

  • % Done changed from 100 to 90

I have code that accomplishes this. I will work on submitting it to the devel list. The only charts that are missed are charts on statistics that do not represent a specific set of hosts, the memory charts.

#3 Updated by Jacob McCann over 7 years ago

Sent my commit to foreman-dev http://groups.google.com/group/foreman-dev/t/52ce84ac65e25f13 for review.

#4 Updated by Amos Benari over 7 years ago

  • Status changed from New to Closed
  • % Done changed from 90 to 100

#5 Updated by Jacob McCann over 7 years ago

Line 78 of charts.js has 2 '~' before 'VAL1'. Maybe that is wanted/needed, but I'm thinking it may be a typo?

Also available in: Atom PDF