-
Notifications
You must be signed in to change notification settings - Fork 186
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
Search: Add folder search #1215
Search: Add folder search #1215
Conversation
The previous search implementation allowed to search for items (using the Filtering is apparently possible using the emitted events (which is currently undocumented): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good for me
0d51d28
to
8f511d5
Compare
Signed-off-by: Sean Molenaar <sean@seanmolenaar.eu>
8f511d5
to
1541d7d
Compare
Codecov Report
@@ Coverage Diff @@
## master #1215 +/- ##
============================================
- Coverage 87.25% 86.51% -0.74%
- Complexity 701 708 +7
============================================
Files 60 61 +1
Lines 2519 2537 +18
============================================
- Hits 2198 2195 -3
- Misses 321 342 +21
Continue to review full report at Codecov.
|
Changed - Add BATS as integration tests (#1213) - Update FeedFetcher to import categories from feeds (#1248) - Update serialization of item to include categories (#1248) - Make PHPStan stricter (#955) - Search: Add folder search (#1215) - Improve test coverage (#1263) - Allow directly adding a feed without going through the discovery process (#1265) Fixed - Do not show deleted feeds in item list (#1214) - Fix update queries (#1211)
Changed - Add BATS as integration tests (#1213) - Update FeedFetcher to import categories from feeds (#1248) - Update serialization of item to include categories (#1248) - Make PHPStan stricter (#955) - Search: Add folder search (#1215) - Improve test coverage (#1263) - Allow directly adding a feed without going through the discovery process (#1265) Fixed - Do not show deleted feeds in item list (#1214) - Fix update queries (#1211) Signed-off-by: Benjamin Brahmer <info@b-brahmer.de>
Changed - Add BATS as integration tests (#1213) - Update FeedFetcher to import categories from feeds (#1248) - Update serialization of item to include categories (#1248) - Make PHPStan stricter (#955) - Search: Add folder search (#1215) - Improve test coverage (#1263) - Allow directly adding a feed without going through the discovery process (#1265) Fixed - Do not show deleted feeds in item list (#1214) - Fix update queries (#1211) Signed-off-by: Benjamin Brahmer <info@b-brahmer.de>
* Search: Add folder search Signed-off-by: Sean Molenaar <sean@seanmolenaar.eu>
Changed - Add BATS as integration tests (nextcloud#1213) - Update FeedFetcher to import categories from feeds (nextcloud#1248) - Update serialization of item to include categories (nextcloud#1248) - Make PHPStan stricter (nextcloud#955) - Search: Add folder search (nextcloud#1215) - Improve test coverage (nextcloud#1263) - Allow directly adding a feed without going through the discovery process (nextcloud#1265) Fixed - Do not show deleted feeds in item list (nextcloud#1214) - Fix update queries (nextcloud#1211) Signed-off-by: Benjamin Brahmer <info@b-brahmer.de>
This adds folder search to the unified search.
@nextcloud/news and @nextcloud/designers any opinions on what should be searchable in news? I'm currently thinking just feeds and folders.