Skip to content

Added #17494: Advanced-search and predefined filters #18264

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 681 commits into
base: develop
Choose a base branch
from
Open

Added #17494: Advanced-search and predefined filters #18264

wants to merge 681 commits into from
+21,494 −128,348

Conversation

@cz-lucas
Copy link

@cz-lucas cz-lucas commented Nov 28, 2025

Added #17494: Advanced-search and predefined filters

New functionality

Advanced search:

  • Added new advanced-search which allows filtering on backend
  • Added functionality to manage private and public filters. The provided new permissions for new functionality
  • Create new settings-page to manage filters

API:

  • Added API to manage the predefined filters

Database:

  • Added tables for predefined filters and predefined filter permissions

Testing:

  • Added plenty of unit- and feature tests for the advanced search and predefined filters
  • Added tests for the "old" advanced search

Screenshots

Advanced search:
image
Predefined filter create modal:
image
Predefined filter settings page
image

cz-lucas and others added 30 commits October 28, 2025 08:18
@cz-lucas
Fixed LDAP testcase
a71a2f5
@cz-lucas
Merge pull request #63 from imbus/rebase-20251028
13ded3f 
Rebase 20251028
@cz-lucas
Merge branch 'bugfix-date-filter-input' into pre-develop-Lc
fbe3ee1
@cz-lucas
Merge remote-tracking branch 'origin/refactor-SIT-99_ByFilter' into p…
8c07fef 
…re-develop-Lc
@cz-lucas
Merge branch 'copilot/update-advanced-search-dropdown' into pre-devel…
132e393 
…op-Lc
@cz-lucas
Merge pull request #60 from imbus/more-minor-ui-stuff
07c9299 
More minor UI stuff
@cz-lucas
Fixed failing testcases
8394f82
@cz-lucas
Merge pull request #64 from imbus/pre-develop-Lc
7a80ed2 
Pre develop lc -> develop
@cz-lucas
Fixed that some dropdowns are expanded even ones that shouldn't
279aefe
@cz-lucas
Fixed that placeholder won't be shown completly in the dropdowns
09c0ac8
@cz-lucas
Fixed failing datequery-tests
3c8a4f7
@cz-lucas
Fixed failing datequery-tests
21c9ec9
@cz-lucas
Skipping all date-query tests
b660128
@cz-lucas
Skipping all date-query tests
3a4694d
@cz-lucas
Skipping all date-query tests
17b87d9
@cz-lucas
Merge pull request #66 from imbus/fix-taling-datequery-tests
d67d8fd 
Fixed failing datequery-tests
@cz-lucas
Merge pull request #65 from imbus/bugfix-all-dropdowns-are-expanded
ae7c8f2 
Bugfix all dropdowns are expanded
@cz-lucas
Merge remote-tracking branch 'upstream/develop' into rebase-20251030
02aa72c
@cz-lucas
Merge pull request #67 from imbus/rebase-20251030
7c0372c 
Rebase 20251030
@cz-lucas
Fixed that daterange is sent twice to the backend in a filter
7633955
@cz-lucas
Fixed datefilter
0ef6004
@cz-lucas
Fixed datequerytests
0addcf6
@cz-lucas
Fixed tests
a811a2f
@cz-lucas
Merge pull request #69 from imbus/bugfix-date-filter-is-broken
cb0c55b 
Bugfix date filter is broken
@cz-lucas
Fixed that filter operators wont be applied on textfields
6680864
@cz-lucas
Merge pull request #70 from imbus/bugfix-filters-operators-wont-be-ap…
858fccc 
…plied

Fixed that filter operators wont be applied on textfields
@cz-lucas
Fixed that duplicate name warning is always shown when creating a filter
3007419
@cz-lucas
Skipping failing tests in pipeline
4e51fa5
@Klausw-1
Data query tests fixed
8a0b53e
@Klausw-1
filterservice fix
fc42a7b
cz-lucas and others added 28 commits December 2, 2025 10:24
@cz-lucas
Renamed tests
cf82c13
@cz-lucas
Some more minor changes
106a93e
@cz-lucas
Merge pull request #121 from imbus/rename-testcases
5bc8034 
Rename testcases
@JonasGoseberg
run npm
44fbccf
@JonasGoseberg
removed unused code
c7b86aa
@cz-lucas
And more code-style fixes
2a30ea2
@JonasGoseberg
refactored some testclasses
234da5a
@cz-lucas
Decreased code-complxity
bf193f7
@cz-lucas
Fixed missing function
9b7d764
@JonasGoseberg
Merge pull request #122 from imbus/SIT-202-cleanup-codacy
ee9a325 
fixed generic onject injection sink warning
@cz-lucas
Added
9bb0c70
@cz-lucas
Merge branch 'pre-merge' into and-more-codestyle-improvements
01171a2
@cz-lucas
Merge pull request #123 from imbus/and-more-codestyle-improvements
3212a89 
And more codestyle improvements
@cz-lucas
Fix comprehensibility issues
7dce4aa
@cz-lucas
Merge remote-tracking branch 'upstream/develop' into more-minor-impro…
5a1d992 
…vements
@cz-lucas
More formatting stuff later...
fe2030c
@JonasGoseberg
next round of codacy improvements
916aea1
@JonasGoseberg
npm run prod
70517a3
@JonasGoseberg
Merge pull request #124 from imbus/SIT-202-more-codacy-cleanup
a8bd4ec 
Sit 202 more codacy cleanup
@cz-lucas
Added escaping
77c9e9a
@cz-lucas
Merge branch 'pre-merge' of github.com:imbus/snipe-it into more-minor…
8422362 
…-improvements
@cz-lucas
Fixed unresolved merged conflict
f2a5f4c
@cz-lucas
Merge pull request #125 from imbus/more-minor-improvements
4a28351 
More minor improvements
@JonasGoseberg
hotfix error on modal open
d7ae9e4
@cz-lucas
Fixed some more very minor issues
dd259cb
@cz-lucas
Merge pull request #126 from imbus/hotfix-modal-open
3bcbfdc 
hotfix error on modal open
@cz-lucas
Merge from upstream/develop
9b02a52
@cz-lucas
Fixed some more minor stuff
fe98dc7
@cz-lucas cz-lucas marked this pull request as ready for review December 3, 2025 09:41
@cz-lucas cz-lucas requested a review from snipe as a code owner December 3, 2025 09:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@snipe snipe Awaiting requested review from snipe snipe is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@cz-lucas @Klausw-1 @JonasGoseberg