Skip to content
This repository has been archived by the owner on Mar 27, 2024. It is now read-only.

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

Open
aguywithanidea opened this issue Sep 29, 2017 · 1 comment
Assignees

Comments

@aguywithanidea
Copy link
Collaborator

aguywithanidea commented Sep 29, 2017

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 aguywithanidea self-assigned this Sep 29, 2017
@aguywithanidea aguywithanidea added this to the 2.5.0 milestone Sep 29, 2017
@aguywithanidea aguywithanidea modified the milestones: 2.5.0, 2.6.1 Nov 15, 2017
@aguywithanidea aguywithanidea removed this from the 2.6.1 milestone Dec 18, 2017
@aguywithanidea
Copy link
Collaborator Author

**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.

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

No branches or pull requests

1 participant