Bug #11821
closed
[API Server] Update discovery document to include latest API updates
0%
Description
Two new calls are missing:
- include trash
- recursive listing of group contents
Updated by Radhika Chippada over 7 years ago
- Assigned To set to Radhika Chippada
Updated by Tom Clegg over 7 years ago
These need to be added to Colection._index_requires_parameters etc. (see source:services/api/app/controllers/arvados/v1/groups_controller.rb)
Updated by Radhika Chippada over 7 years ago
- Status changed from New to In Progress
Updated by Radhika Chippada over 7 years ago
Branch 11821-discovery-doc-updates is ready for review @ cac20b7dd2177edebc0a4f8087032d1143bd524e
Test run @ https://ci.curoverse.com/job/developer-run-tests/323/
Updated by Lucas Di Pentima over 7 years ago
Updates at cac20b7dd2177edebc0a4f8087032d1143bd524e LGTM, thanks.
Updated by Radhika Chippada over 7 years ago
- Status changed from In Progress to Resolved
Applied in changeset arvados|commit:d66b42e150030edff5ddb211d76d03d4bcd572ec.
Updated by Radhika Chippada over 7 years ago
- Status changed from Resolved to In Progress
The arv command seems to send include_trash = "false" sometimes. This will be misinterpreted as include_trash != nil. Updated to check for bogus values @ 82c90381c1ebdab5be7870b13ca8898e81b042c8
Updated by Lucas Di Pentima over 7 years ago
Ran services/api local tests successfully.
LGTM, thanks!
Updated by Radhika Chippada over 7 years ago
- Status changed from In Progress to Resolved
I have not seen this issue after my initial observation and logic exists in application_controller to treat "false" as false. So, I am going to discard this update and leave it as in cac20b7dd2177edebc0a4f8087032d1143bd524e.