-
Notifications
You must be signed in to change notification settings - Fork 219
Conversation
The release ZIP for this PR is accessible via:
Script Dependencies ReportThere is no changed script dependency between this branch and trunk. This comment was automatically generated by the TypeScript Errors Report
🎉 🎉 This PR does not introduce new TS errors. |
Size Change: +1.73 kB (0%) Total Size: 1.08 MB
ℹ️ View Unchanged
|
This PR has been marked as If deemed still relevant, the pr can be kept active by ensuring it's up to date with the main branch and removing the stale label. |
… 404 error on the frontend
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.
Given that this PR is very similar to #8157, I took the liberty to do a code review here as well. 🙂
It's testing great and code looks good, I just left a few comments inline.
Co-authored-by: Albert Juhé Lluveras <[email protected]>
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, thanks for updating the PR, @albarin!
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.
The block is testing well. The style is working as expected. I left some minor comments, otherwise, this is 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.
Just some more minor comments and we can merge this PR.
Co-authored-by: Tung Du <[email protected]>
Co-authored-by: Tung Du <[email protected]>
The goal of this PR is to add the new Product Results Count block as part of the blockifying the product archive template project.
It shows the total number of products and how many of them are displayed at the moment based on the global query.
It uses the
woocommerce_result_count()
function to render the block on the front end.In the Editor it displays "Showing 1-X of X results".
Fixes #8060
Screenshots
Testing
Automated Tests
User Facing Testing
Expected: Product Results Count block is not available there
Expected: It displays "Showing 1-X of X results".
Classic Template
and addProducts
blockProducts
block hasInherit query from template
setting enabled (!)Expected: It displays "Showing 1-16 of 17 results". - depends on number of products your store has, numbers may differ, but it displays correct number of product that shows up in
Products
blockProducts
Expected: It displays "Showing 17-17 of 17 results". - depends on number of products your store has, numbers may differ, but it displays correct number of product that shows up in
Products
blockWooCommerce Visibility
Changelog