)]}'
{"api-ref/source/v3/volumes-v3-snapshots.inc":[{"author":{"_account_id":27248,"name":"Prateek Goel","email":"prateek.goel2@tatacommunications.com","username":"prateek.goel"},"change_message_id":"125a13cb66020e51fc7becab5ded14fc6d00b72a","unresolved":false,"context_lines":[{"line_number":426,"context_line":""},{"line_number":427,"context_line":"Preconditions"},{"line_number":428,"context_line":""},{"line_number":429,"context_line":"-Volume status must be ``available`` or ``error``"},{"line_number":430,"context_line":""},{"line_number":431,"context_line":"Normal response codes: 202"},{"line_number":432,"context_line":""}],"source_content_type":"text/x-c++src","patch_set":1,"id":"3fa0c359_8cabb31b","line":429,"range":{"start_line":429,"start_character":0,"end_line":429,"end_character":49},"updated":"2018-02-19 17:05:22.000000000","message":"Please add a space between \"-\" and text.","commit_id":"a8efd00430faa657f58af13d8b318c8beddf455b"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"7cb4b90711d63f0fbecc5a489ad4473185a8f569","unresolved":false,"context_lines":[{"line_number":426,"context_line":""},{"line_number":427,"context_line":"Preconditions"},{"line_number":428,"context_line":""},{"line_number":429,"context_line":"    -Volume status must be ``available`` or ``error``"},{"line_number":430,"context_line":""},{"line_number":431,"context_line":"Normal response codes: 202"},{"line_number":432,"context_line":""}],"source_content_type":"text/x-c++src","patch_set":2,"id":"3fa0c359_fcc2a533","line":429,"updated":"2018-02-25 10:53:34.000000000","message":"This is not properly formatted for RST. I think you need to put \u0027:\u0027 at the end of \u0027Preconditions\u0027, unindent this line, and have a space after \u0027-\u0027.","commit_id":"dea4555045d14521369edb9c34a748170d26c66c"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"9a170c1481da7b92c2ff44b6167a8e6ad0cf9535","unresolved":false,"context_lines":[{"line_number":472,"context_line":""},{"line_number":473,"context_line":".. rest_status_code:: success ../status.yaml"},{"line_number":474,"context_line":""},{"line_number":475,"context_line":"   - 202"},{"line_number":476,"context_line":""},{"line_number":477,"context_line":""},{"line_number":478,"context_line":"Request"}],"source_content_type":"text/x-c++src","patch_set":4,"id":"df7087c5_df158f3e","line":475,"updated":"2018-03-23 14:19:43.000000000","message":"Is this supposed to just list the response code on success, or also likely failure codes?","commit_id":"d8f39f83a4844c2714b347bf5c5ef436f459826b"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"ea4da6d073c604abbfb6614fc958b9adb727e8d2","unresolved":false,"context_lines":[{"line_number":472,"context_line":""},{"line_number":473,"context_line":".. rest_status_code:: success ../status.yaml"},{"line_number":474,"context_line":""},{"line_number":475,"context_line":"   - 202"},{"line_number":476,"context_line":""},{"line_number":477,"context_line":""},{"line_number":478,"context_line":"Request"}],"source_content_type":"text/x-c++src","patch_set":4,"id":"df7087c5_5f42df10","line":475,"in_reply_to":"df7087c5_df158f3e","updated":"2018-03-23 14:25:35.000000000","message":"It should have error codes too. That was not in the docs when switching over to the rest_status_code method, but does need to be added at some point.\n\nI had the same thought - it seems kind of strange now that we are listing some preconditions, but this doesn\u0027t really give any indication of what the response would be if those preconditions are not met.","commit_id":"d8f39f83a4844c2714b347bf5c5ef436f459826b"}]}