Skip to content

Conversation

@Aljullu
Copy link
Contributor

@Aljullu Aljullu commented Apr 27, 2021

Fixes woocommerce/woocommerce-blocks#4130.
Fixes woocommerce/woocommerce-blocks#4131.
Fixes woocommerce/woocommerce-blocks#4132.

WordPress 5.8 is adding an item_link and item_link_description label properties to custom post types and taxonomies, so they can be used to add friendly names to Navigation Link block variations. Notice those variations are created automatically from CPT and taxonomies.

In order to get those block variations to work properly, we also need to enable Rest for Product Categories and Product Tags so they can be searched using the UI.

Screenshots

Labels

Before After
imatge imatge

Rest suggestions

imatge

How to test the changes in this Pull Request:

  1. You will need to install a recent nightly of WordPress 5.8 (you can use WordPress beta tester for this) and the latest version of Gutenberg.
  2. Create a page and add a Navigation block.
  3. Inside the navigation block, add a product link, a product category link and a product tag link and verify all of them work correctly and have labels that make it clear they are related to products.

Changelog entry

Add labels and allow searching terms in WooCommerce Navigation link block variations

Aljullu added 2 commits April 27, 2021 15:09
This way, they can be searched when adding the Product Category Link
and Product Tag Link blocks.
@Aljullu Aljullu self-assigned this Apr 27, 2021
@Aljullu Aljullu requested a review from frontdevde April 28, 2021 12:53
@Aljullu
Copy link
Contributor Author

Aljullu commented Apr 28, 2021

@frontdevde pinging you for review from our team. We will also need this PR reviewed by somebody from Proton.

Copy link
Contributor

@frontdevde frontdevde left a comment

Choose a reason for hiding this comment

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

Tested as per testing instructions and it worked as advertised. Nice improvement! LGTM 👍

Options Tag recommendation
Screenshot 2021-04-29 at 11 39 20 Screenshot 2021-04-29 at 11 41 02

@Aljullu Aljullu requested a review from ObliviousHarmony May 18, 2021 10:01
Copy link
Contributor

@roykho roykho left a comment

Choose a reason for hiding this comment

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

LGTM! Works as expected. Thanks for the PR.

@roykho roykho merged commit 02981cf into trunk May 19, 2021
@roykho roykho deleted the fix/navigation-block-preparation branch May 19, 2021 18:55
@woocommercebot woocommercebot added release: add changelog Mark all PRs that have not had their changelog entries added. [auto] release: add testing instructions PRs that have not had testing instructions added to the wiki. [auto] labels May 19, 2021
@roykho roykho added the release: cherry-pick Commits from this PR also needs to be added to current release branch. label May 19, 2021
@github-actions github-actions bot added this to the 5.5.0 milestone May 19, 2021
@roykho roykho modified the milestones: 5.5.0, 5.4.0 May 19, 2021
@ObliviousHarmony ObliviousHarmony added changelog added and removed release: cherry-pick Commits from this PR also needs to be added to current release branch. release: add changelog Mark all PRs that have not had their changelog entries added. [auto] release: add testing instructions PRs that have not had testing instructions added to the wiki. [auto] labels May 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Navigation: Create Product Tag Link block Navigation: Create Product Category Link block Navigation: Create Product Link block

6 participants