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 with '_featured' and '_filled' meta on duplicated posts #2839

Closed
wants to merge 3 commits into from

Conversation

fjorgemota
Copy link
Member

@fjorgemota fjorgemota commented Jul 22, 2024

Fixes #2827

Changes Proposed in this Pull Request

  • Centralize add_post_meta to add _filled and _featured in a single place after the post is called.

Testing Instructions

  1. Apply this patch
  2. Install ACF
  3. Install a plugin to duplicate posts, like Yoast Duplicate Post
  4. Create a custom field/form for job listings
  5. Add a new job listing, fill it normally
  6. Go to the job listing page on admin, click to Clone it
  7. Edit the new post, edit the custom field, save, check if it doesn't break everything

Release Notes

  • Fix with '_featured' and '_filled' meta on duplicated posts

New or Updated Hooks and Templates

Deprecated Code

Screenshot / Video


Plugin build for fc3b10c
📦 Download plugin zip
▶️ Open in playground

@fjorgemota fjorgemota self-assigned this Jul 22, 2024
@fjorgemota
Copy link
Member Author

This approach won't work because the way the plugins to duplicate posts work. Closing..

@fjorgemota fjorgemota closed this Jul 23, 2024
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.

Error messages: Updating failed. Could not update the meta value of _filled in database.
1 participant