Iulia Cazan
Iulia Cazan
I can replicate the issue with other plugins as well, not sure if it is WooCommerce related, but rather core 6.7 related. The issue is not that we get the...
@carolinan, thank you for the feedback! Will look into replacing the code with the `get_the_terms()` as you suggested and see if that makes it better.
@carolinan, your idea is better from both perspectives. Since we already get the post terms a few lines above in the code, we can reuse that, and also safer for...
Hi! It's been a while since working on this, so I rebased the code to be up to date with the trunk. It's ready again for picking up the conversation...
Hi! It's been a while since working on this. It's ready again for picking up the conversation if anyone has availability. Thanks!
@carolinan, the branch has been rebased and is ready for resuming the review. Thanks!
 @carolinan, similar attributes have been applied to the block, as per your suggestion. Please review. Thanks!
> If you scroll down to the test results, this PR is failing two tests and it has a merge conflict. These must be resolved. That is a consequence of...
@carolinan, @krstivoja, this is done, just letting you know of the development. My terrible rebase 😅 triggered review requests from more people, it wasn't intentional and I apologize for the...
It's ready again for picking up the conversation if anyone has availability. Thanks!