adobe / aem-core-cif-components

A set of configurations and components to get you started with AEM Commerce development
Apache License 2.0
103 stars 80 forks source link

ProductList component filtering out categories from filter attribute #866

Closed ayushofficial07 closed 2 years ago

ayushofficial07 commented 2 years ago

User Story

As a user, I want to be able to view the categories filter as an available aggregation in a productList page so that I can drill down to different sub-categories from filtering menu also.

Description & Motivation

The current productList CIF component implementation ( https://github.com/adobe/aem-core-cif-components/blob/master/bundles/core/src/main/java/com/adobe/cq/commerce/core/components/internal/models/v1/productlist/ProductListImpl.java ) trims out the category filter in the getSearchResultSet function.

image

As the function uses some private and protected functions and classes, it is very complicated to override this existing functionality, thus having another function that provides the category filter in the aggregations will serve the purpose efficiently.

Deliverables

For providing the categories we can follow two approaches :

Acceptance Criteria

On dragging and dropping a core cif ProductList component, I should be able to see the category filter along with other filters.

Verification Steps

buuhuu commented 2 years ago

This is tracked in CIF-2686.

Unfortunately this is not trivial to implement. On a PLP we would like to link to the individual category pages from the aggregation. For that we need additional parameters (url_key, url_path, uid) which are not returned by the aggregation.

As a limited but feasible scope of this requirement, we will implement the category filter in a way that it only includes the direct children of the PLPs current category. This allows for layered navigation and we can query the required parameters with the category query we anyway do.

Does that address your requirement?

buuhuu commented 2 years ago

This has be resolved as described above in https://github.com/adobe/aem-core-cif-components/pull/921