-
Notifications
You must be signed in to change notification settings - Fork 17
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: hide cpt from menu #1365
fix: hide cpt from menu #1365
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, @dkoo! It seems like this is the current behavior in |
Hey @miguelpeixe, good job getting this PR merged! 🎉 Now, the Please check if this PR needs to be included in the "Upcoming Changes" and "Release Notes" doc. If it doesn't, simply remove the label. If it does, please add an entry to our shared document, with screenshots and testing instructions if applicable, then remove the label. Thank you! ❤️ |
@dkoo, fixed the Segments tab in Automattic/newspack-plugin@7e515b6 |
🎉 This PR is included in version 3.1.5-alpha.1 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
🎉 This PR is included in version 3.1.5 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
All Submissions:
Changes proposed in this Pull Request:
This is a fix for Automattic/newspack-plugin#3549.
Popups post type is already not shown in the menu, but without the explicit flag, WP will override (more specifically here) the
parent_file
filter we're using to activate a custom menu when creating a new prompt:How to test the changes in this Pull Request:
trunk
, navigate to create a new segment (/wp-admin/post-new.php?post_type=newspack_popups_cpt
)Other information: