@WP-API/amigos #reviewmerge

I would like @rmccue to weigh in here and in: #1255, #1254, and #1253

From the HTTP spec for DELETE:

A successful response SHOULD be 200 (OK) if the response includes an entity describing the status, 202 (Accepted) if the action has not yet been enacted, or 204 (No Content) if the action has been enacted but the response does not include an entity.

I don’t know what an entity describing the status means

This generally looks OK to me. One thing we might want to consider is wrapping the post in another object (e.g. { data: {...}, trashed: true, gone: false }) to make it clear that the request did succeed (since we're meant to be describing the status), buuuuut that might be overkill.

Also, should we consider this from #627?

if ( $new_status === 'trash' ) {
    // Send a 202 if it's only trashed to indicate that it will be
    // actually deleted later
    $response->set_status( 202 );

i.e. 200 = permanently deleted, 202 = trashed

Not sure we need the distinction now with the split from double-DELETE to separate trash/perma-delete, but worth thinking about.

Not sure we need the distinction now with the split from double-DELETE to separate trash/perma-delete, but worth thinking about.

See #1260