Skip to content

Add core WooCommerce product_brand to Pinterest feed as g:brand#1173

Open
rbgly wants to merge 1 commit intodevelopfrom
fix/add-product-brand-to-feed
Open

Add core WooCommerce product_brand to Pinterest feed as g:brand#1173
rbgly wants to merge 1 commit intodevelopfrom
fix/add-product-brand-to-feed

Conversation

@rbgly
Copy link
Copy Markdown

@rbgly rbgly commented Apr 23, 2026

Summary

  • Adds g:brand field to the Pinterest XML product feed, sourced from the core WooCommerce product_brand taxonomy (WooCommerce Brands)
  • Variations inherit brand from their parent product, consistent with how g:product_type handles categories
  • A pinterest_for_woocommerce_product_brand filter allows overriding the brand value per product

Closes PIN4WOO-68

Test plan

  • Add a brand via Products → Brands and assign it to a product
  • Trigger a feed sync and inspect the XML — confirm <g:brand>BrandName</g:brand> appears for that product
  • Confirm products with no brand assigned have no <g:brand> element in the feed
  • Confirm a product variation inherits the brand from its parent
  • Run vendor/bin/phpunit --testsuite unit --filter testPropertyBrand — should show 2 passing tests

🤖 Generated with Claude Code

Reads the first term from the product_brand taxonomy (WooCommerce Brands)
and outputs it as <g:brand> in the XML feed. Variations inherit the brand
from their parent product. A pinterest_for_woocommerce_product_brand filter
allows overriding the value per product.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@github-actions github-actions Bot added changelog: fix Took care of something that wasn't working. type: bug The issue is a confirmed bug. labels Apr 23, 2026
@wildrivia wildrivia requested a review from louwie17 April 24, 2026 09:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog: fix Took care of something that wasn't working. type: bug The issue is a confirmed bug.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant