-
Notifications
You must be signed in to change notification settings - Fork 215
Create the Products by Attribute template
#7660
Conversation
|
The release ZIP for this PR is accessible via: |
e2bbdc4 to
b3cb445
Compare
TypeScript Errors ReportFiles with errors: 432
assets/js/atomic/blocks/product-elements/image/test/block.test.js
assets/js/data/payment/check-payment-methods.ts |
|
Size Change: -61 B (0%) Total Size: 991 kB
ℹ️ View Unchanged
|
Script Dependencies ReportThere is no changed script dependency between this branch and trunk. This comment was automatically generated by the |
b4ff3fd to
9ff0c72
Compare
9ff0c72 to
adecf91
Compare
gigitux
left a comment
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.
I tested the PR and it works correctly! Great job!
Instead of using taxonomy-pa, I think that it is more clear if we use a name like taxonomy-product_attribute. What do you think?
gigitux
left a comment
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.
Thanks for updating the PR! Great work! 💪
LGTM!
thealexandrelara
left a comment
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.
I reviewed and tested it and everything looks great. Great work! LGTM!
The goal of this PR is to add a new
Products by Attributetemplate.Fixes #7647
Testing
Automated Tests
User Facing Testing
Browse all templates(or/wp-admin/site-editor.php?postType=wp_template).Products by Attributetemplate shows on the list.Products>Attribute(/wp-admin/edit.php?post_type=product&page=product_attributes).Enable archives?checkbox, and save.Configure termson the attribute you just edited it.Viewon one of the terms to view it in the frontend.Product by Attributetemplate.WooCommerce Visibility
Changelog