-
Notifications
You must be signed in to change notification settings - Fork 7
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
Add a search builder that allows use of a new request handler #3819
Conversation
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.
Just some thoughts...
supporting a different Solr RequestHandler Co-authored-by: Anna Headley <hackartisan@users.noreply.github.com> Co-authored-by: Carolyn Cole <carolyncole@users.noreply.github.com> Co-authored-by: Eliot Jordan <eliotjordan@users.noreply.github.com> Co-authored-by: James R. Griffin III <jrgriffiniii@users.noreply.github.com> Co-authored-by: Trey Pendragon <tpendragon@users.noreply.github.com>
20ecd07
to
d9c8466
Compare
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.
Thanks, team 1! Just a question that came to mind while I was reading up on what the qt
parameter means 🤣
# When adding a new Ranking Algorithm the name will need to be added to this list before it can be utilized | ||
# For example if you added a cats ranking algorithm, with a CatsSearchBuilder you would add "cats" to this list ["engineering","cats"] | ||
# This is to make sure the user can not just execute any SearchBuilder in the system | ||
def allowed_search_algorithms |
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.
Thanks for adding this check!
…algortihm swapping This shows that the other two features take presidence Co-authored-by: James R. Griffin III <jrgriffiniii@users.noreply.github.com>
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.
Thanks, team 1!
Based on a querystring parameter, and also behind a feature flipper.
closes #3808
requires pulibrary/pul_solr#375