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

Update filter removal to match Post Kinds v3.2.6 #24

Merged
merged 2 commits into from May 16, 2019

Conversation

Projects
None yet
3 participants
@MrKapowski
Copy link
Contributor

commented May 16, 2019

The latest version of Post Kinds (v3.2.6) changed the priority of filters applied to content, from 20 to 9. This resulted in remove_filter no longer working as intended and double output of post kinds information. Changing the priority to 9 fixes the problem.

MrKapowski added some commits May 16, 2019

Update filter removal to match latest Post Kinds
The latest version of Post Kinds (v3.2.6) changed the priority the filters are applied to content, from 20 to 9. This resulted in remove_filter no longer working as intended. Changing the priority to 9 fixes the problem.
Merge pull request #1 from MrKapowski/post-kind-filters
Update filter removal to match latest Post Kinds
@dshanske

This comment has been minimized.

Copy link

commented May 16, 2019

Would you prefer I put in a theme support flag for Post Kinds? So this would not happen in future?

@pfefferle

This comment has been minimized.

Copy link
Owner

commented May 16, 2019

thanks @MrKapowski

@dshanske would make things easier

@pfefferle pfefferle merged commit d087d57 into pfefferle:master May 16, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.