)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":37843,"name":"Olamide Ojo","display_name":"olamidepeterojo","email":"peterojoolamide@gmail.com","username":"olamidepeterojo"},"change_message_id":"67687d30c8a2b7012ae50315cb8fb27a6dd60e7b","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"7234f137_e9550d64","updated":"2026-10-20 18:56:43.000000000","message":"Hello Adeola, this is nice work! I just have an inline suggestion","commit_id":"acc3a4ec75dfed800b1f9529ef46743121b87770"},{"author":{"_account_id":37860,"name":"Adeola Adesina","email":"adeolaadesina46@gmail.com","username":"AdeolaAdesina"},"change_message_id":"5003fe7ce33dc5d473e9a8fcf38f676853d445c6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"a5e22ac4_a029f2be","in_reply_to":"7234f137_e9550d64","updated":"2026-10-20 19:38:47.000000000","message":"Done","commit_id":"acc3a4ec75dfed800b1f9529ef46743121b87770"}],"manila/api/schemas/share_access_metadata.py":[{"author":{"_account_id":37843,"name":"Olamide Ojo","display_name":"olamidepeterojo","email":"peterojoolamide@gmail.com","username":"olamidepeterojo"},"change_message_id":"82adba73d1ee589c9469710939596143742a836d","unresolved":true,"context_lines":[{"line_number":43,"context_line":"    \u0027additionalProperties\u0027: False"},{"line_number":44,"context_line":"    }"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"delete_response_body \u003d {"},{"line_number":47,"context_line":"    \u0027type\u0027: \u0027null\u0027,"},{"line_number":48,"context_line":"    }"}],"source_content_type":"text/x-python","patch_set":3,"id":"d2fa093c_542dfce6","line":46,"updated":"2026-10-20 19:10:41.000000000","message":"Here, the delete endpoint currently doesn\u0027t have a response body, so I\u0027m suggesting a better approach would be delete_response instead of delete_response_body","commit_id":"acc3a4ec75dfed800b1f9529ef46743121b87770"},{"author":{"_account_id":37860,"name":"Adeola Adesina","email":"adeolaadesina46@gmail.com","username":"AdeolaAdesina"},"change_message_id":"5003fe7ce33dc5d473e9a8fcf38f676853d445c6","unresolved":false,"context_lines":[{"line_number":43,"context_line":"    \u0027additionalProperties\u0027: False"},{"line_number":44,"context_line":"    }"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"delete_response_body \u003d {"},{"line_number":47,"context_line":"    \u0027type\u0027: \u0027null\u0027,"},{"line_number":48,"context_line":"    }"}],"source_content_type":"text/x-python","patch_set":3,"id":"5cd676b1_d57b71a0","line":46,"in_reply_to":"d2fa093c_542dfce6","updated":"2026-10-20 19:38:47.000000000","message":"Thank you for the observation. The delete endpoint does not have a response body. This is why I put the response body as \u0027type: null\u0027, it ensures no data is returned in error. I chose to leave it as delete_response_body because the type specified addresed the response body and also ensure the naming convection is in sync","commit_id":"acc3a4ec75dfed800b1f9529ef46743121b87770"}]}