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

Get "Include pre-publish checklist" status #376

Closed
htmgarcia opened this issue Apr 27, 2022 · 4 comments
Closed

Get "Include pre-publish checklist" status #376

htmgarcia opened this issue Apr 27, 2022 · 4 comments

Comments

@htmgarcia
Copy link
Contributor

This may help when included in WP core WordPress/gutenberg#34405
Screen Shot 2022-04-27 at 9 35 46

Involved code: https://github.com/publishpress/PublishPress-Checklists/blob/development/modules/checklists/assets/js/meta-box.js#L682-L693

Issue related somehow to #343

@olatechpro
Copy link
Collaborator

@htmgarcia Can you explain what this issue is all about or what we're looking to achieve? I really don't understand what do do or what to test for on this issue

@htmgarcia
Copy link
Contributor Author

@olatechpro I don't remember the details, but I think is related to the fact that if "Include pre-publish" is disabled, the checklists from PP Checklist requirements are ignored when saving the post (e.g. post must have 2 tags). So we would need a way to detect if "Include pre-publish" is enabled or disabled as a workaround for this issue https://wordpress.org/support/topic/last-update-disables-the-publish-button-2/

@olatechpro
Copy link
Collaborator

Oh, thanks. I'll test towards that @htmgarcia

@olatechpro
Copy link
Collaborator

olatechpro commented Jan 9, 2024

This is no longer needed due to introduction of Checklists Panel #567

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants