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
Fix wrong Shop title shown in classic themes after deleting the page #46429
Fix wrong Shop title shown in classic themes after deleting the page #46429
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Hi @samueljseay, Apart from reviewing the code changes, please make sure to review the testing instructions as well. You can follow this guide to find out what good testing instructions should look like: |
…46429) * Fix wrong Shop title shown in classic themes after deleting the page * Add changelog file * Add explanatory comment
* Fix wrong Shop title shown in classic themes after deleting the page (#46429) * Fix wrong Shop title shown in classic themes after deleting the page * Add changelog file * Add explanatory comment * Prep for cherry pick 46429 --------- Co-authored-by: Albert Juhé Lluveras <contact@albertjuhe.com> Co-authored-by: WooCommerce Bot <no-reply@woo.com>
Changes proposed in this Pull Request:
This PR partially reverts some changes introduced in #44537.
Rationale: we had some code that made sure the Shop page title was Shop. While we initially wrote that code to be applied only to block themes, given that
BlockTemplatesController
was run also in classic themes, it caused the change to apply to classic themes as well. When we moved that logic to theProductCatalogTemplate
class in #44537, it stopped being run in classic themes, causing the regression.This PR moves the code back to
BlockTemplatesController
to make sure it runs in classic themes as well.This is a temporary solution. If this code is expected to run in classic and block themes, ideally it shouldn't be in
BlockTemplatesController
.Kudos to @jorgeatorres for finding the root of the issue!
How to test the changes in this Pull Request:
/shop
).<title>
tags, that means the title that appears in the tab bar.Changelog entry
Significance
Type
Message
Comment