Skip to content

Add hooks: actionFacetedSearchSetSupportedControllers actionFacetedSearchFilters actionFacetedSearchCacheKeyGeneration #2066

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
Open
Codencode wants to merge 1 commit into
base: 9.x
Choose a base branch
from
Open

Add hooks: actionFacetedSearchSetSupportedControllers actionFacetedSearchFilters actionFacetedSearchCacheKeyGeneration #2066

Codencode wants to merge 1 commit into from

Conversation

@Codencode
Copy link

@Codencode Codencode commented Dec 7, 2025 •
edited
Loading

Questions Answers
Branch? 9.x
Description? This PR adds documentation for the three new hooks in the ps_facetedsearch module:
- actionFacetedSearchSetSupportedControllers
- actionFacetedSearchFilters
- actionFacetedSearchCacheKeyGeneration
Fixed ticket?
Related PR PrestaShop/ps_facetedsearch#1170 - PrestaShop/ps_facetedsearch#1195
Sponsor company Codencode snc

@Codencode
Add hooks: actionFacetedSearchSetSupportedControllers actionFacetedSe…
dffb6c8 
…archFilters actionFacetedSearchCacheKeyGeneration
@github-actions github-actions bot added the 9.x label Dec 7, 2025
@Codencode
Copy link
Author

Codencode commented Dec 7, 2025

@kpodemski
Hi Krystian, as you suggested, I've created the documentation for the hooks I added to the ps_facetedsearch module.
Could you please check if everything is okay?
Keep in mind this is my first time creating a PR for documentation, so it's likely something might be missing or not entirely correct.

Thanks!

@Codencode Codencode requested a review from kpodemski December 7, 2025 14:43
@kpodemski kpodemski self-assigned this Dec 9, 2025
@kpodemski
Copy link
Contributor

kpodemski commented Dec 9, 2025

thank you Paolo, I'll check it as soon as I can

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

Reviewers

@kpodemski kpodemski Awaiting requested review from kpodemski

At least 1 approving review is required to merge this pull request.

Assignees

@kpodemski kpodemski

Labels

9.x

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Codencode @kpodemski