Skip to content
This repository has been archived by the owner on Aug 8, 2020. It is now read-only.

Fix search-page #141

Open
ghost opened this issue May 1, 2019 · 3 comments
Open

Fix search-page #141

ghost opened this issue May 1, 2019 · 3 comments
Assignees
Labels
enhancement New feature or request
Projects
Milestone

Comments

@ghost
Copy link

ghost commented May 1, 2019

The search-page, at the moment, is only searching for the title.

We need a clean implementation of a universal search-EP and call it from the frontend, where we can provide every field offered for our search-query.

@ghost ghost added the bug A programmer messed something up label May 1, 2019
@ghost ghost added this to the v0.3-alpha milestone May 1, 2019
@Mondei1 Mondei1 added enhancement New feature or request and removed bug A programmer messed something up labels May 15, 2019
@Mondei1
Copy link
Member

Mondei1 commented May 15, 2019

Are you sure this is a bug? I don't think so. I'll changed it.

@Mondei1
Copy link
Member

Mondei1 commented May 15, 2019

I think what you mean is implement the other option-where-* fields in the header? The universal EP is the "intelligent search". I'll add option-where-number-primary, option-where-number-secondary and option-where-fileextension for now.

@Mondei1 Mondei1 self-assigned this May 15, 2019
@Mondei1 Mondei1 added this to In progress in Programming May 15, 2019
@ghost
Copy link
Author

ghost commented May 16, 2019

Uh... yeah you are right, that was misleading.

This EP is made for letting the user specify in detail, what attributes he wants to add to the search query.

This is NOT meant to be the "universal-search-EP" where the user just inputs some string and the backend tries to make sense of it and applies the splitted substrings into the fitting query-fields :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
Programming
  
In progress
Development

Successfully merging a pull request may close this issue.

1 participant