Project

General

Profile

Bug #24351

Chart items on the dashboard are no longer links

Added by Ewoud Kohl van Wijngaarden about 3 years ago. Updated 5 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Dashboard
Target version:
Difficulty:
Triaged:
No
Bugzilla link:

Description

A long time ago the segments in the charts were clickable. For example in the Host Configuration Chart for Puppet you can have x% in out of sync. That used to be clickable and take you to the search to show those hosts. The table with the same data still has clickable links.


Related issues

Related to Foreman - Bug #31526: WebUI spinner spins forever when clicking a column within config reportClosed
Blocked by Foreman - Refactor #32009: Remove flot_pie_chartClosed

Associated revisions

Revision befa5578 (diff)
Added by Ondřej Ezr 5 months ago

Fixes #24351 - chart click through search

Fixes long lasting regression of charts on dashboard not being clickable through.
Adds posibility to specify custom filters for chart segments.
Moves click handler from DonutChartService to ChartService, as there is no reason for
this funcionality not to be added to more chart types in the future.

Revision 1982799b (diff)
Added by Ondřej Ezr 5 months ago

Refs #24351 - dashboard removals revert

Reverts accidental removal of method from befa557.
We are still using the old search_links method.

History

#1 Updated by The Foreman Bot 9 months ago

  • Assignee set to Ondřej Ezr
  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/8210 added

#2 Updated by Ondřej Ezr 9 months ago

  • Related to Bug #31526: WebUI spinner spins forever when clicking a column within config report added

#3 Updated by Ondřej Ezr 7 months ago

#4 Updated by Ondřej Ezr 5 months ago

  • Target version set to 2.5.0

#5 Updated by The Foreman Bot 5 months ago

  • Fixed in Releases 2.5.0 added

#6 Updated by Ondřej Ezr 5 months ago

  • Status changed from Ready For Testing to Closed

#7 Updated by The Foreman Bot 5 months ago

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

Also available in: Atom PDF