Feature #370

Add "interesting" filter to reports

Added by Paul Kelly almost 4 years ago. Updated almost 4 years ago.

Status:ClosedStart date:08/30/2010
Priority:NormalDue date:
Assigned To:Paul Kelly% Done:

100%

Category:Reporting
Target version:0.1-6
Difficulty: Bugzilla link:
Found in release: Pull request:
Story points-
Velocity based estimate-

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 e53706e8
Added by Paul Kelly almost 4 years ago

Fixes #370 - Added 'interesting' filter to reports

History

#1 Updated by Paul Kelly almost 4 years 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

#2 Updated by Ohad Levy almost 4 years 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.

#3 Updated by Paul Kelly almost 4 years 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?

#4 Updated by Paul Kelly almost 4 years ago

rebased
tested

#5 Updated by Paul Kelly almost 4 years ago

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

#6 Updated by Ohad Levy almost 4 years ago

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

#7 Updated by Ohad Levy almost 4 years ago

  • Status changed from Ready For Testing to Assigned

can you please rebase it again?

thanks

#8 Updated by Paul Kelly almost 4 years ago

  • Status changed from Assigned to Ready For Testing

Rebased and tested

#9 Updated by Paul Kelly almost 4 years ago

  • % Done changed from 0 to 100

#10 Updated by Ohad Levy almost 4 years ago

  • Status changed from Ready For Testing to Closed

Also available in: Atom PDF