HeyPublisher / heypublisher-submission-manager

HeyPublisher Wordpress Plugin
GNU General Public License v2.0
2 stars 1 forks source link

Editor should have to double-confirm before rejecting accepted piece #71

Open aguywithanidea opened 7 years ago

aguywithanidea commented 7 years ago

Having issues where an accepted piece is rejected with one click. Need to update the plugin so that it make the editor confirm before rejecting under this situation. They need to be aware that it will purge the submission from the database and will not be available.

Same with "withdrawn" status when already accepted.

Additionally - not all of these statuses are valid after a work has been accepted:

image

aguywithanidea commented 3 years ago

Note:

The function reject_submission in HeyPublisher::Page::Submissions class calls WP function wp_delete_post which will force deletion of an already-accepted submission.

The wp_delete_post function is also called by the function withdraw_submission in the same class when an editor marks the submission as having been "withdrawn".

There is currently not an audit trail for editors to know when a submission has been removed from pending posts due to either of these actions. When fixing this, there should also be an audit trail introduced. This will require server code to support.