-
Notifications
You must be signed in to change notification settings - Fork 0
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
Require Unpublish Date #1
Comments
I meant to mention that in order for this to work, we will need to use the Required Fields branch. |
Got some follow-up questions:
|
Not network enabled: only for our announcement set up. Required for the 'slides' post type. Currently unpublish is the only requirement. |
I've written a small plugin called "Announcements Publishing Checklist". Activate it on the sites you need to use it on and ensure that both "Publishing Checklist" and "Unpublish" plugins are activated. Next, try creating a slide and see if the checklist works as expected. I tried this locally on the |
Do I need the Fusion Publishing Checklist, or just yours and Unpublish. If the Fusion one, is it the "required fields" branch? |
You need to install Fusion's "Publishing Checklist" plugin (using the |
It looks right, but I'm still able to get the slide published without an UnPublish Date. You can find it up and running here: https://announcementstemplate.commons.hwdsb.on.ca/wp-admin |
I think the issue is the Let me look into this a bit more. |
Okay, should work now. Forgot to sync up my changes to production! Can you try again and let me know if it works? |
Now the button defaults to Submit for Review, and even though I've met the requirements, the post goes as Draft, not Published. |
Try a new post. You have to set the Unpublish date, then hit "Submit for Review" and then hit "Publish" again. That's the way the Publishing Checklist |
Ok. I see that now. https://github.com/fusioneng/publishing-checklist/pull/33#issuecomment-186585382 It would be nice if the button could be changed to say something like "Check Requirements", then Publish. Submit for Review makes it seem like perhaps we have ripped the ability to publish away from users. |
I've added a JS snippet that dynamically switches out the "Submit for Review" label for "Check Requirements". Let me know if that is okay with you. I'll comment on that PR once we like these changes. |
That's perfect. Now I just need to find a way to replace the clunky datepicker in Unpublish with something a bit more user friendly. Maybe this one: https://wordpress.org/plugins/publish-date-datepicker/ |
New Issue: https://github.com/hwdsbcommons/HWDSB-Commons/issues/116 Closing this as solved. Looks great. |
That's a side effect of using the Publishing Checklist plugin. You first have to complete all required tasks. In our case, set an Unpublish date and hit "Check Requirements". This should fulfill the requirements and you'll have access to the Publish field again where you can set the publish date into the future. |
We use this plugin to allow users to Unpublish content on our Announcements site.
https://github.com/humanmade/Unpublish
Although it's there, many users don't bother to set a date, leaving the Announcement admin having to clean up old posts. Requiring an Unpublish date would be amazing.
The text was updated successfully, but these errors were encountered: