Added filters to gives possibility to specify which term to use to gener... #758

Merged
merged 1 commit into from Dec 24, 2012

Conversation

Projects
None yet
3 participants
Contributor

annzaan commented Dec 21, 2012

...ate permalink/breadcrumb for product

Optart's customer wanted to have the category in breadcrumb, which user used to show the product. Right now if product is in many categories the "first" one is taken and used to generate breadcrumb and permalink for product. My plugin puts to permalink the category in which user is currently, then the breadcrumb is dispalyed on basis of that category.

Contributor

divergeinfinity commented Dec 24, 2012

@chriscct7 -- same for this one Chris, the commit itself is fine and I'm going to ignore the failed build notice and merge it in.

Is it just something with Travis that for some reason it's taking to long to execute the tests?

Why would the PHP 5.3 test fail and the PHP 5.4 test pass for multisite? (or so it appears?)

@divergeinfinity divergeinfinity pushed a commit that referenced this pull request Dec 24, 2012

Jeff Parsons Merge pull request #758 from annzaan/breadcrumb_filters
Added filters to gives possibility to specify which term to use to gener...
5192c3a

@divergeinfinity divergeinfinity merged commit 5192c3a into jigoshop:dev Dec 24, 2012

1 check failed

default The Travis build failed
Details
Member

chriscct7 commented Dec 24, 2012

@divergeinfinity Sometimes the merge of dependencies fails. I'm working on an optomized script.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment