Cleaner way to submit/cancel documents via REST

Refer: https://github.com/frappe/frappe_docs/pull/221

Often in the community, this is recommended for cancelling/submitting:

PUT /api/resource/doctype/name 
{"docstatus": 2}

If you read document.py this isn’t really equivalent to cancelling the document. IMO the current best method is to supply run_method (both submit/cancel are whitelisted).

Current method that works well is:

POST /api/resource/:doctype/:name
# body
{ "run_method": "submit" }

That’s ugly and returns null 💩

Proposals

1. Fix document.py so just sending PUT request with new docstatus works. (this we should do regardless of # 2)
2. Provide a cleaner API for submit/cancel (or all methods that work with run_method)?

I suggested something like this in docs PR:

POST /api/resource/doctype/name/cancel

cc: @gavindsouza @surajshetty3416 @netchampfaris @sagarvora @barredterra

[ankush]

@ankush i would rather disallow naming series containing /. This causes lots of other issue, for example when switching between form and print view.

Sadly lots of users do use naming series with / in them.

Maybe name part of URL should be urlencoded (/ -> %2F) @surajshetty3416 ?

[ankush]

Proposal of POST /api/resource/doctype/name/cancel will have to be scrapped because I can’t think of any decent way to split URL that will respect naming series containing / 🥲

We can hardcode few methods like cancel/submit maybe? split URL and check if last part matches one of the supported method?

Sent PR for fixing rest of the issues.