Skip to content
This repository has been archived by the owner on Nov 16, 2024. It is now read-only.

support multiple taxonomies #15

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

cherbst
Copy link

@cherbst cherbst commented Dec 5, 2016

Allows multiple taxonomies for groups. Adds a filter for themes/plugins to override taxonomies.

@imath
Copy link
Owner

imath commented Dec 8, 2016

Wow @cherbst, your pull request looks pretty nice! Thanks a lot, will look at it more carfully really soon.

before, if the taxonomy slug also occured in the term slug,
then the term slug was also prefixed with the bp group slug
@imath
Copy link
Owner

imath commented Dec 23, 2016

I'm sorry i forgot to commit a fix for BuddyPress 2.7. I'm going to commit it. So this will probably have an impact on this pull request.

Copy link
Owner

@imath imath left a comment

Choose a reason for hiding this comment

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

After a quick look, i think it's important to make sure your changes are also working on network installs. And according to this line https://github.com/imath/bp-groups-taxo/pull/15/files?diff=unified#diff-8d79d3ae70c6906dd6d01c479fccbe91R112 it doesn't seem to be the case. Could you give another look?

@danimalweb
Copy link

I'd love the see this PR merged. Can it be fixed?

@tripod3d tripod3d mentioned this pull request Nov 12, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants