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

Search Block: count results per facet #4766

Open
1 task
avoinea opened this issue May 15, 2023 · 7 comments
Open
1 task

Search Block: count results per facet #4766

avoinea opened this issue May 15, 2023 · 7 comments
Assignees
Milestone

Comments

@avoinea
Copy link
Member

avoinea commented May 15, 2023

In eea.facetednavigation we had a feature that would allow one to enable/disable result count per facet. This is also a major feature of modern search solutions like ElasticSearch or Solr.

Would be great to be able to enable counting per facet within Volto Search Block.

For this we will need:

SOLR API: https://solr.apache.org/guide/8_1/json-facet-api.html#bucketing-facet-example
ES API: https://www.elastic.co/guide/en/app-search/current/facets.html

Plone Classic Faceted Navigation counter implementation:

Faceted navigation example from Plone Classic:

Screenshot 2023-05-15 at 11 21 22

@erral
Copy link
Member

erral commented May 15, 2023

cc: @ionlizarazu

@avoinea avoinea added this to the Plone 6.1 milestone Jun 6, 2023
@sneridagh
Copy link
Member

@avoinea what's the status for this one? It's ok to slate it for 18?

@avoinea
Copy link
Member Author

avoinea commented Sep 14, 2023

@sneridagh AFAIK there is no progress on it lately. Thus, I think it's safer to move it to 18.x.
cc @razvanMiu

@davisagli
Copy link
Member

@sneridagh This is currently more of a proof-of-concept. It should wait for 18. Ideally, we should improve the API in ZCatalog/plone.app.querystring so that we don't have to duplicate code from them.

@sneridagh sneridagh modified the milestones: Plone 6.1, 18.x.x Sep 26, 2023
@FarooqAlaulddin
Copy link

Any updates on this feature? Any advice of how to temporality support it?

@avoinea
Copy link
Member Author

avoinea commented Nov 17, 2023

@FarooqAlaulddin There is an open PR on plone.restapi plone/plone.restapi#1637 that has to be refactored and move code upstream directly in ZCatalog and plone.app.querystring as @davisagli suggested 😉

@sneridagh sneridagh modified the milestones: 18.x.x, 19.x.x May 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants