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.

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

Polish search #4094

Merged
merged 4 commits into from
Jul 6, 2022
Merged

Polish search #4094

merged 4 commits into from
Jul 6, 2022

Conversation

aduffeck
Copy link
Contributor

@aduffeck aduffeck commented Jul 5, 2022

This PR improves feedback when providing invalid queries and adds support for limiting the number of results returned.

Fixes #4016
Fixes #4017

@ownclouders
Copy link
Contributor

ownclouders commented Jul 5, 2022 •
edited
Loading

💥 Acceptance test cs3ApiTests-ocis failed. Further test are cancelled...

@sonarcloud
Copy link

sonarcloud bot commented Jul 5, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@aduffeck aduffeck merged commit 24fbb4a into owncloud:master Jul 6, 2022
@aduffeck aduffeck deleted the polish-search branch July 6, 2022 06:44
ownclouders pushed a commit that referenced this pull request Jul 6, 2022
@aduffeck
commit 24fbb4a
6d62cf6 
Merge: c24a8cd c5ec476
Author: Andre Duffeck <aduffeck@users.noreply.github.com>
Date:   Wed Jul 6 08:44:43 2022 +0200

    Merge pull request #4094 from aduffeck/polish-search

    Polish search
@micbar micbar mentioned this pull request Jul 19, 2022
Closed
36 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@C0rby C0rby C0rby approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@aduffeck @ownclouders @C0rby