Skip to content

[ISSUE-555] Add old hooks from classic theme#556

Merged
ga-devfront merged 1 commit intoPrestaShop:developfrom
tblivet:issue-555
Oct 18, 2023
Merged

[ISSUE-555] Add old hooks from classic theme#556
ga-devfront merged 1 commit intoPrestaShop:developfrom
tblivet:issue-555

Conversation

@tblivet
Copy link
Contributor

@tblivet tblivet commented Oct 17, 2023

Questions Answers
Description? Add old classic theme hooks. The displaySearch hook has not been added because in classic theme he was only used on 404 error page and if we add it on this page this will generate duplicate ID on the page. #555
Type? improvement
BC breaks? no
Deprecations?
Fixed ticket? Fixes #555
Sponsor company @PrestaShopCorp
How to test? These hooks are just called in the code

Copy link
Contributor

@ga-devfront ga-devfront left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank for your work I think it can be merged then the QA is ok.

@Robin-Fischer-PS
Copy link

It can/should be tested by a dev :)
Thanks for the PR @tblivet !

@ga-devfront ga-devfront merged commit fe47af7 into PrestaShop:develop Oct 18, 2023
@leemyongpakvn leemyongpakvn added this to the Beta milestone Dec 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

Three hooks don't work on the hummingbird theme

8 participants