Feature #370

Add "interesting" filter to reports

Added by Paul Kelly over 1 year ago. Updated over 1 year ago.

Status:Closed Start:08/30/2010
Priority:Normal Due date:
Assigned to:Paul Kelly % Done:

100%

Category:Reporting
Target version:0.1-6
Backlog:No Difficulity:
Votes: 0

Description

Provide a filter on the reports search line that limits the list to reports that contain changes.

This constraint should also be the default for the link on the reports page for a host. That is, a host's list of reports should only contain those of interest.

Associated revisions

Revision e53706e8cd42a5ed025aaefaf7cb244c1174a789
Added by Paul Kelly over 1 year ago

Fixes #370 - Added 'interesting' filter to reports

History

Updated by Paul Kelly over 1 year ago

  • Category set to Reporting
  • Status changed from New to Ready For Testing
  • Assigned to set to Paul Kelly
  • Branch set to feature/370-interesting-report-filter

Updated by Ohad Levy over 1 year ago

  • Target version set to 0.1-6
I've just tried this branch, two comments:
  1. it doesn't work - the box is selected, but the reports are not correct (you see all reports, not just interesting ones).
  2. I'm not sure that I want to users not to be able to see the good reports, if I understood correctly, your patch will not allow us to see the good reports in the list.

Updated by Paul Kelly over 1 year ago

1) There is a misunderstanding here. When you select the reports page you get ALL reports. If you access search you are presented with a search builder with the interesting filter activated by default. You can remove that tick if you wish.
2) The user is able to see whatever reports that they want via the search facility. Maybe we should go the whole way, (as you thought,) and set the interesting flag for reports on the reports front page and not just in the search line. The user could then see all reports just by unticking the interesting flag on the search line and clicking GO. If this were the case though, you would only be given a list of hosts that had something to report and not all hosts; but maybe this is better anyway. What do you think?

Updated by Paul Kelly over 1 year ago

rebased
tested

Updated by Paul Kelly over 1 year ago

The reports heading now reflects whether the list is for all reports or for eventful reports
Rebased, tested

Updated by Ohad Levy over 1 year ago

Can you please revisit this ticket after the changes #405?

Updated by Ohad Levy over 1 year ago

  • Status changed from Ready For Testing to Assigned

can you please rebase it again?

thanks

Updated by Paul Kelly over 1 year ago

  • Status changed from Assigned to Ready For Testing

Rebased and tested

Updated by Paul Kelly over 1 year ago

  • % Done changed from 0 to 100

Updated by Ohad Levy over 1 year ago

  • Status changed from Ready For Testing to Closed
  • Backlog set to No

Also available in: Atom PDF