Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement service state filter for quicksearch widget #713

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

moschlar
Copy link
Contributor

General information

I use the quicksearch sidebar widget a lot and I often find myself wanting to see all services matching a description that are not OK - so I thought it would be nice to have an additional filter so that I can query for e.g. s:Certificate st:1 st:2.

Proposed changes

I implemented a ServiceStateMatchPlugin class to accomplish this - it works quite well now!

Open tasks/questions:

  • What is the first element of the return value of get_matches supposed to be?
  • How to correctly display the "preselected" quicksearch filters in the graphical filter selection dialog?

(See previous PR at #587)

@moschlar moschlar force-pushed the quicksearch-state branch from 35da0bd to c8f5960 Compare June 20, 2024 12:46
Copy link

github-actions bot commented Jul 5, 2024

This PR is stale because it has been open for 14 days with no activity and the Github Actions are not passing.

@github-actions github-actions bot added the Stale Scheduled for auto-close label Jul 5, 2024
@moschlar

This comment was marked as resolved.

@github-actions github-actions bot removed the Stale Scheduled for auto-close label Jul 6, 2024
Copy link

This PR is stale because it has been open for 14 days with no activity and the Github Actions are not passing.

@github-actions github-actions bot added Stale Scheduled for auto-close and removed Stale Scheduled for auto-close labels Jul 20, 2024
@moschlar moschlar force-pushed the quicksearch-state branch 5 times, most recently from 27c8d96 to e00ed14 Compare July 29, 2024 07:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants