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

Add custom taxonomy support #13

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

Conversation

idflood
Copy link

@idflood idflood commented Oct 2, 2020

What does this implement/fix?

As discussed here this adds support for custom taxonomy. I took the same structure and way of doing things as the custom "Post" types so it should be consistent with this.

Does this close any currently open issues?

Yes, it should fixes Rareloop/lumberjack#16

Any relevant logs, error output, etc?

No :) and all added test are ✔

Any other comments?

We will add Lumberjack to our base theme and taxonomies is a must. Let me know if I need to make some changes, I would be more than happy to help with this feature : )

And once it's merged I will make the required changes to document it here with a pull request https://github.com/Rareloop/lumberjack-docs

@coveralls
Copy link

Coverage Status

Coverage increased (+0.004%) to 88.785% when pulling 4134412 on idflood:features/add-custom-taxonomy-support into 46105b8 on Rareloop:master.

5 similar comments
@coveralls
Copy link

Coverage Status

Coverage increased (+0.004%) to 88.785% when pulling 4134412 on idflood:features/add-custom-taxonomy-support into 46105b8 on Rareloop:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.004%) to 88.785% when pulling 4134412 on idflood:features/add-custom-taxonomy-support into 46105b8 on Rareloop:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.004%) to 88.785% when pulling 4134412 on idflood:features/add-custom-taxonomy-support into 46105b8 on Rareloop:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.004%) to 88.785% when pulling 4134412 on idflood:features/add-custom-taxonomy-support into 46105b8 on Rareloop:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.004%) to 88.785% when pulling 4134412 on idflood:features/add-custom-taxonomy-support into 46105b8 on Rareloop:master.

@coveralls
Copy link

coveralls commented Oct 2, 2020

Coverage Status

Coverage increased (+0.02%) to 88.797% when pulling 92c9382 on idflood:features/add-custom-taxonomy-support into 46105b8 on Rareloop:master.

@claudelebo
Copy link

Hi, when it will be merged to the core ? thank you

@daltonrooney
Copy link

I'm also interested to know if it will be merged at some point.

@Androlax2
Copy link

I am also interested to know when it will be merged.

@AlexSBlackburn
Copy link

AlexSBlackburn commented Oct 25, 2021

Is there a timeline for when this will be merged? I would like to use Lumberjack professionally but this is quite a large missing feature which makes me reluctant to implement it in my workplace just yet.

@nealoke
Copy link

nealoke commented Oct 29, 2021

+1 on this merge request, any idea when this could be implemented?

@arnaudbagnis
Copy link

🆙

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.

Custom Taxonomy support in Lumberjack
8 participants