)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":33648,"name":"Nahim Alves de Souza","email":"nahimsouza@outlook.com","username":"nahimsouza"},"change_message_id":"5466b925848bbb1fa262153313bc25bfdfc74d99","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"6e07eb07_a34f7480","updated":"2021-12-02 20:05:31.000000000","message":"Thank you Lucas, the patch looks good to me.","commit_id":"c39f529a6153223547475a254b86ce2952fed3f1"},{"author":{"_account_id":33431,"name":"Fábio Oliveira","email":"fabioaurelio1269@gmail.com","username":"fabiooliveira1"},"change_message_id":"bbf5f6870b297e99f82d6c49d0ca512eb2f6b6e0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"55d33163_4c129eea","updated":"2022-01-21 14:11:03.000000000","message":"LGTM. Thank you Lucas for working on this issue :)\n","commit_id":"f0ea6d04ba07d4628b29d0f917270808a3b89cdc"},{"author":{"_account_id":33986,"name":"Lucas Morais de oliveira","email":"lucasmoliveira059@gmail.com","username":"lucasmoliveira059"},"change_message_id":"2a877a74f72d8f1a4a0a9cdb79dfdac9cb2cb064","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"865eabb5_bf5a43cb","updated":"2022-01-21 13:15:34.000000000","message":"Thank You all for reviewing!\nHere is a new patch set for you to take a look \u003d)","commit_id":"f0ea6d04ba07d4628b29d0f917270808a3b89cdc"},{"author":{"_account_id":33986,"name":"Lucas Morais de oliveira","email":"lucasmoliveira059@gmail.com","username":"lucasmoliveira059"},"change_message_id":"d2ce89622ab58543695972cfa1eff79b668c292d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"f40c0303_e35e3463","updated":"2022-01-24 14:50:41.000000000","message":"Thank you all for reviewing!","commit_id":"f0ea6d04ba07d4628b29d0f917270808a3b89cdc"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"b93aa51ca191e49135f417c6e6c1b6489a87d4a3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"8f718a3a_cf19fa57","updated":"2022-01-21 21:39:46.000000000","message":"Thanks for posting this patch, and sorry it\u0027s been sitting so long.  An improvement noted inline.","commit_id":"f0ea6d04ba07d4628b29d0f917270808a3b89cdc"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"41d365e4def8a196f63541ee012e9f9baa059f76","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"80eae9f3_22483252","updated":"2022-01-21 14:39:04.000000000","message":"Thanks! I guess we could use the same pattern to fix other similar definitions (like links), but that\u0027s definitely out of scope for this patch.","commit_id":"f0ea6d04ba07d4628b29d0f917270808a3b89cdc"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"f6ae31d0c05769998bb20b509250df1527890af9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"fafa1993_062418a9","updated":"2022-03-18 19:46:20.000000000","message":"Ninja-approving a documentation change.","commit_id":"1f67b918ac9c770c5ba68aea95ac84914a1283c6"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"4f05a1d8f7610ab5af33cc57c8dba13e0964c0de","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"7178d41c_a3111733","updated":"2022-02-18 14:15:35.000000000","message":"The changes requested in the comments were implemented.","commit_id":"1f67b918ac9c770c5ba68aea95ac84914a1283c6"}],"api-ref/source/v3/parameters.yaml":[{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"0e05092c5ae127a7684d79708c19d2835e247626","unresolved":true,"context_lines":[{"line_number":666,"context_line":"  min_version: 3.47"},{"line_number":667,"context_line":"backup_links_optional:"},{"line_number":668,"context_line":"  description: |"},{"line_number":669,"context_line":"    The backup links."},{"line_number":670,"context_line":"  in: body"},{"line_number":671,"context_line":"  required: false"},{"line_number":672,"context_line":"  type: array"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5797200f_047e60e0","line":669,"updated":"2021-12-17 14:30:15.000000000","message":"Shouldn\u0027t this explain a bit more what this is about (which I\u0027m not sure either, but I haven\u0027t checked the code)","commit_id":"c39f529a6153223547475a254b86ce2952fed3f1"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"1d3175b723481bdf3de813d077e4d9452efd7f9c","unresolved":true,"context_lines":[{"line_number":666,"context_line":"  min_version: 3.47"},{"line_number":667,"context_line":"backup_links_optional:"},{"line_number":668,"context_line":"  description: |"},{"line_number":669,"context_line":"    The backup links."},{"line_number":670,"context_line":"  in: body"},{"line_number":671,"context_line":"  required: false"},{"line_number":672,"context_line":"  type: array"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"d917bafa_3ca7bdd6","line":669,"in_reply_to":"5797200f_047e60e0","updated":"2021-12-17 14:52:44.000000000","message":"Agree. However, to be fair L1658 links_vol_optional says nothing relevant. Eager to see more comments on this.","commit_id":"c39f529a6153223547475a254b86ce2952fed3f1"},{"author":{"_account_id":33986,"name":"Lucas Morais de oliveira","email":"lucasmoliveira059@gmail.com","username":"lucasmoliveira059"},"change_message_id":"2a877a74f72d8f1a4a0a9cdb79dfdac9cb2cb064","unresolved":false,"context_lines":[{"line_number":666,"context_line":"  min_version: 3.47"},{"line_number":667,"context_line":"backup_links_optional:"},{"line_number":668,"context_line":"  description: |"},{"line_number":669,"context_line":"    The backup links."},{"line_number":670,"context_line":"  in: body"},{"line_number":671,"context_line":"  required: false"},{"line_number":672,"context_line":"  type: array"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"b395f5d7_5c5693f3","line":669,"in_reply_to":"d917bafa_3ca7bdd6","updated":"2022-01-21 13:15:34.000000000","message":"As you said I used the links_vol_optional as an example and most links items are similar, but I have already updated the backup_links_optional description. Hope is good now \u003d)","commit_id":"c39f529a6153223547475a254b86ce2952fed3f1"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"b93aa51ca191e49135f417c6e6c1b6489a87d4a3","unresolved":true,"context_lines":[{"line_number":666,"context_line":"  min_version: 3.47"},{"line_number":667,"context_line":"backup_links_optional:"},{"line_number":668,"context_line":"  description: |"},{"line_number":669,"context_line":"    An array containing a dict with the full ``href`` to a backup and it\u0027s ``rel``."},{"line_number":670,"context_line":"  in: body"},{"line_number":671,"context_line":"  required: false"},{"line_number":672,"context_line":"  type: array"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"41367ce5_37127dde","line":669,"range":{"start_line":669,"start_character":54,"end_line":669,"end_character":65},"updated":"2022-01-21 21:39:46.000000000","message":"That\u0027s not quite correct.  The backup_links show up when the number of backups exceed the requested (or default) page size, and the href points to the next page of results, not to a backup.  Also, JSON doesn\u0027t have a dict data type (that\u0027s a Python thing).\n\nI think you are right that we need to mention the ``href`` and ``rel`` in the description, especially because they don\u0027t show up in the example.  So maybe something like:\n\nAn array containing an object with the following fields: ``\"rel\"`` with the value ``\"next\"`` and ``href``, whose value is a link to the next page of backups.  Only appears when there are more backups than are listed in the current response.","commit_id":"f0ea6d04ba07d4628b29d0f917270808a3b89cdc"},{"author":{"_account_id":33986,"name":"Lucas Morais de oliveira","email":"lucasmoliveira059@gmail.com","username":"lucasmoliveira059"},"change_message_id":"d2ce89622ab58543695972cfa1eff79b668c292d","unresolved":false,"context_lines":[{"line_number":666,"context_line":"  min_version: 3.47"},{"line_number":667,"context_line":"backup_links_optional:"},{"line_number":668,"context_line":"  description: |"},{"line_number":669,"context_line":"    An array containing a dict with the full ``href`` to a backup and it\u0027s ``rel``."},{"line_number":670,"context_line":"  in: body"},{"line_number":671,"context_line":"  required: false"},{"line_number":672,"context_line":"  type: array"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"8abbf826_835153da","line":669,"range":{"start_line":669,"start_character":54,"end_line":669,"end_character":65},"in_reply_to":"41367ce5_37127dde","updated":"2022-01-24 14:50:41.000000000","message":"Done. The description is way better now!","commit_id":"f0ea6d04ba07d4628b29d0f917270808a3b89cdc"}]}