-
Notifications
You must be signed in to change notification settings - Fork 10.7k
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Create update_template_data util and formatting improvements #46408
Conversation
Hi @WunderBart, @nefeline, 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: |
Hi @nefeline, 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: |
91d1c60
to
b47c085
Compare
2121efc
to
b47c085
Compare
b47c085
to
31839e2
Compare
cfadf3c
to
40911ba
Compare
40911ba
to
2c3a629
Compare
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.
Changes test well locally, thanks! 馃殌
* Create update_template_data util * Fix comment typo * Formatting * Remove unnecessary use of BlockTemplateUtils * Add changelog file * Create update_template_data util (II)
Changes proposed in this Pull Request:
This PR includes some improvements from #44778. The reason is that #44778 is blocked by upstream work so it's unlikely to be merged soon, so with this PR we will bring to
trunk
what's currently "mergeable". 馃檪Specifically, this PR includes these changes:
update_template_data
util to avoid code duplication.BlockTemplateUtils
.How to test the changes in this Pull Request:
This PR doesn't include any functionality changes, but some basic smoke testing of block templates is appreciated.
Go to Appearance > Editor > Templates.
Make some edits to some templates. Verify you can save them and they preserve the template name on save. Ie: the top bar should show
Single Product
instead ofsingle-product
:Visit the templates in the frontend and verify the edits have been applied successfully.
Changelog entry
Significance
Type
Message
Comment