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

Considere an empty query search as a placeholder search #916

Merged
merged 3 commits into from
Sep 28, 2020

Conversation

qdequele
Copy link
Member

@qdequele qdequele commented Aug 14, 2020

Fix #856; Relative tracking issue: #729

@MarinPostma
Copy link
Contributor

Nice rust-fmt haha

@qdequele
Copy link
Member Author

I now use VSCode with a rust-analyzer 😉

@Kerollmops
Copy link
Member

Could you please split the rustfmt thing from the core change you've done?

It would be easier to review.

@qdequele qdequele force-pushed the placeholder-search-on-empty-string branch from 64a3498 to 7430776 Compare August 17, 2020 15:43
@ManyTheFish ManyTheFish added this to the 10/2020 milestone Aug 31, 2020
@qdequele qdequele marked this pull request as ready for review September 11, 2020 12:15
meilisearch-http/src/routes/search.rs Outdated Show resolved Hide resolved
Kerollmops
Kerollmops previously approved these changes Sep 25, 2020
@Kerollmops
Copy link
Member

bors r+

@Kerollmops
Copy link
Member

@qdequele Whoops there is a merge conflict, could you rebase, please?

@bors
Copy link
Contributor

bors bot commented Sep 25, 2020

Merge conflict.

Copy link
Contributor

@MarinPostma MarinPostma left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

bors r+

@bors
Copy link
Contributor

bors bot commented Sep 28, 2020

Build succeeded:

@bors bors bot merged commit 099a080 into master Sep 28, 2020
@bors bors bot deleted the placeholder-search-on-empty-string branch September 28, 2020 13:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Placeholder search when query is empty does not work
5 participants