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

Fix process_tags overwriting existing tags #154

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

lemacarl
Copy link

@lemacarl lemacarl commented Mar 6, 2018

Currently when one tries to add tags to a post by hooking into the o2_create_post the tags are overwritten by process_tags which overwrites any existing tags.

@mdawaffe
Copy link
Member

Thanks for the suggestion!

With this change, tags cannot be removed by deleting the #inline-tags in the o2 post editor.

Rather that changing o2 (which has complicated tag handling), is it possible to make your code work by hooking into o2_writeapi_post_created instead of o2_create_post?

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.

2 participants