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

Reflect changes in the search service for OC 16. #22

Draft

tgloeggl wants to merge 1 commit into master from oc-16

Member

tgloeggl commented May 17, 2024 •

edited

Remove lucene search, does not exists anymore
Update allowed sort params and their spelling
Update tests

Some things need to be discussed:

Is it necessary to have the api versioned or is graceful degration enough (to get it to work with oc 16 AND before)?

There also seems to be a bug in OC which needs to be addressed first (the sort order currently only works with title)

Fixes #21


          remove lucene search, update sort params and tests

3bac9bf

tgloeggl added the enhancement label

tgloeggl requested a review from ferishili

May 17, 2024 07:54

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment