REST API: using DELETE on /api/v1/shows/ID does not work

It seems the deletion of single shows is not implemented in the REST API. I can delete shows from the Django backend but when I submit a DELETE request to /api/v1/shows/9 (or using any other existing show id), I just receive the show object in the response body and the show still exists afterwards.

Ok, it does work, but it is not utterly clear to me why it works when I do it as an axios request in dashboard but does not work in Postman, unless I add a closing '/' to the URL.

See the following three screenshots as examples. Details below.

Successful delete request from the dashboard:

Unsuccessful delete request from Postman:

Successful delete request from Postman:

In the dashboard request's response data we can see that the request was on http://127.0.0.1:8000/api/v1/shows/12 (the config.url parameter) but the request.responseURL parameter includes a trailing /. In Postman, when I don't include a trailing / in the URL, I can repeatedly make a DELETE request (in this case with a new show with the id 13) and repeatedly get a 200 response with the show object as response body. Only when I use the trailing /, I get a 204 response with empty body, similar to what I get in my request from dashboard.

So, for my client coding this is fine. But in general I think we should change this behaviour, because without the trailing / the DELETE request seems to just work like a GET request. That should not be the case, imo. It leads to unnecessary long debugging and potentially buggy code.

We better don't really delete any show for documentation purposes, but only set is_active to False. Let's kick the ability to delete via API.

mentioned in commit 862486a5

closed

changed milestone to %1.0-alpha2 — Precise Pheasant