Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make product category URLs heirarchical #581

Merged

Conversation

pixeltrix
Copy link
Contributor

Heirarchical URLs for custom taxonomies is an opt-in feature added
in WordPress 3.1. This commit enables that support by adding the
appropriate flag to the rewrite setting for the product_cat taxonomy.

It may be preferred to make this a setting for backwards compatibility,
if so just say where you want it added.

Heirarchical URLs for custom taxonomies is an opt-in feature added
in WordPress 3.1. This commit enables that support by adding the
appropriate flag to the rewrite setting for the product_cat taxonomy.
@GeertDD
Copy link
Contributor

GeertDD commented Feb 6, 2012

+1

@mikejolley
Copy link
Member

Ah cool :)

mikejolley added a commit that referenced this pull request Feb 6, 2012
@mikejolley mikejolley merged commit 862f751 into woocommerce:master Feb 6, 2012
@mikejolley
Copy link
Member

Is this working for you? Mine are still flat after permalink save. Just wondering if it actually made a difference to your cats.

@pixeltrix
Copy link
Contributor Author

It did eventually, I think that the urls may be being cached by the WordPress Transients API. Here's a URL on the site I'm working on: http://woodfields.sites.pixeltrix.co.uk/shop/category/gamecover-seed/other-crops/

@mikejolley
Copy link
Member

Does the 'category' instead of 'product-cat' not conflict with post cats?

@pixeltrix
Copy link
Contributor Author

Blog posts, categories and tags are under /blog so everything is fine.

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.

None yet

3 participants