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

records: optimize search #422

Merged
merged 1 commit into from
Jul 19, 2021
Merged

records: optimize search #422

merged 1 commit into from
Jul 19, 2021

Conversation

sebdeleze
Copy link

  • Adds a parameter in routing to enable or disable search if query is empty.
  • Gets the aggregation only if the facet is expanded or has a filter selected.
  • Loads buckets for an aggregation when a facet is expanded.

Co-Authored-by: Sébastien Délèze sebastien.deleze@rero.ch

@iGormilhit iGormilhit added this to the v1.4.4 milestone Jul 19, 2021
@sebdeleze sebdeleze marked this pull request as ready for review July 19, 2021 13:15
@sebdeleze sebdeleze requested review from jma and zannkukai July 19, 2021 13:15
* Adds a parameter in routing to enable or disable search if query is empty.
* Gets the aggregation only if the facet is expanded or has a filter selected.
* Loads buckets for an aggregation when a facet is expanded.

Co-Authored-by: Sébastien Délèze <sebastien.deleze@rero.ch>
@sebdeleze sebdeleze merged commit a7f87fe into rero:dev Jul 19, 2021
@sebdeleze sebdeleze deleted the sed-optimize-facets branch July 19, 2021 17:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants