)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"3e69f51fb327aa046343d0cf200df967ed251c4e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"4a0e6685_bcff3df1","updated":"2025-09-09 09:03:09.000000000","message":"Here is the screenshot\n- older screenshot without this feature: https://i.imgur.com/UOaMjSB.png\n- Newer screenshot with this feature: https://i.imgur.com/L195yDW.png","commit_id":"c91a423ac35771fc8c58e262fb212404b64d1a88"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"35edad98008eeaab221b46a3b82a5bd5636f9bd0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"a697f7f7_11dd44ec","updated":"2025-09-17 12:55:30.000000000","message":"This works fine and makes the job.\n\nIn some strategies, which have many parameters, it creates a big text box (i.e. the workload stabilization). From usability PoV, I\u0027m not sure if it would be better to limit the lines and getting a scroll bar to move across the spec. Said that, I really don\u0027t know much about UX recommendations, tbh.","commit_id":"37207e38e08fc4f185a8c6288dce2937b5000b72"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"249ad548f4335f3a36a72eebfdac425a35c4f29d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"d9cf66fe_ce5e3586","in_reply_to":"236a0707_45210453","updated":"2025-09-18 04:26:07.000000000","message":"Thank you @amoralej@redhat.com for the review and idea, It sounds fine by me, But I would like to achieve the same as a follow up.","commit_id":"37207e38e08fc4f185a8c6288dce2937b5000b72"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"37391b68d7981db150b32d7b04ca5d8157d44846","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"3cf0af6a_0c02a2a5","in_reply_to":"6927c69a_a41bc2af","updated":"2025-09-19 14:55:31.000000000","message":"Going to W+1 this one, and chandan can propose a follow up with both. Since you also +1 this change.","commit_id":"37207e38e08fc4f185a8c6288dce2937b5000b72"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"a736eab77b9856880e706fcd96a579003e453a31","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"236a0707_45210453","in_reply_to":"a697f7f7_11dd44ec","updated":"2025-09-17 13:00:24.000000000","message":"Also, it\u0027d be nice if we could make some syntax highlighting of the spec in some easy way (i.e. using different color or formats for key, value, elements in a list...)","commit_id":"37207e38e08fc4f185a8c6288dce2937b5000b72"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"df4e22e0f92225ca89c187f96a3187b5851d0f9c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"6927c69a_a41bc2af","in_reply_to":"d9cf66fe_ce5e3586","updated":"2025-09-19 08:41:34.000000000","message":"For the spec box size, i think it\u0027s simply enough to include it in this one (see my comment), if you are ok with that.\n\nFor the spec prettify i agreee that it\u0027s probably better to do a follow up, I don\u0027t have any idea about easy implementation and having even a non-pretty one is better than nothing.","commit_id":"37207e38e08fc4f185a8c6288dce2937b5000b72"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"37391b68d7981db150b32d7b04ca5d8157d44846","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"6cf80b65_6d4e789e","updated":"2025-09-19 14:55:31.000000000","message":"Thanks Chandan","commit_id":"6de76239c598c1c67f89ca11dc648ce1d6863cca"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"f30f019f800739acb90a03a5ec6cf2559d057ac8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"4aef1a4f_fe73e55b","updated":"2025-09-19 14:52:41.000000000","message":"lgtm, the output look good here:\n\nhttps://imgur.com/a/MoZyTO6","commit_id":"6de76239c598c1c67f89ca11dc648ce1d6863cca"}],"watcher_dashboard/content/strategies/views.py":[{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"f30f019f800739acb90a03a5ec6cf2559d057ac8","unresolved":true,"context_lines":[{"line_number":130,"context_line":"                    return obj"},{"line_number":131,"context_line":""},{"line_number":132,"context_line":"            # If JSON schema-like, extract properties for readability"},{"line_number":133,"context_line":"            if isinstance(obj, dict) and \u0027properties\u0027 in obj \\"},{"line_number":134,"context_line":"                    and isinstance(obj.get(\u0027properties\u0027), dict):"},{"line_number":135,"context_line":"                obj \u003d obj[\u0027properties\u0027]"},{"line_number":136,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"b58eb075_3e5ee52f","line":133,"range":{"start_line":133,"start_character":61,"end_line":133,"end_character":62},"updated":"2025-09-19 14:52:41.000000000","message":"avoid using backslash, better to use parentheses intead [1]\n\n[1] https://docs.openstack.org/hacking/latest/user/hacking.html#general","commit_id":"6de76239c598c1c67f89ca11dc648ce1d6863cca"}],"watcher_dashboard/templates/infra_optim/strategies/details.html":[{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"df4e22e0f92225ca89c187f96a3187b5851d0f9c","unresolved":true,"context_lines":[{"line_number":23,"context_line":"        \u003cdt\u003e{% trans \"Update At\" %}\u003c/dt\u003e"},{"line_number":24,"context_line":"        \u003cdd\u003e{{ strategy.updated_at|parse_isotime|default:_(\"-\") }}\u003c/dd\u003e"},{"line_number":25,"context_line":"        \u003cdt\u003e{% trans \"Parameters Spec\" %}\u003c/dt\u003e"},{"line_number":26,"context_line":"        \u003cdd\u003e"},{"line_number":27,"context_line":"          {% if strategy_parameters_spec_pretty %}"},{"line_number":28,"context_line":"            {{ strategy_parameters_spec_pretty|safe }}"},{"line_number":29,"context_line":"          {% else %}"}],"source_content_type":"text/html","patch_set":3,"id":"a5d5c1b0_b8daedd6","line":26,"range":{"start_line":26,"start_character":8,"end_line":26,"end_character":12},"updated":"2025-09-19 08:41:34.000000000","message":"You can adjust size and behavior for this box just by doing:\n\n```\n\u003cdd style\u003d\"height: auto; max-height: 500px; overflow: auto\"\u003e\n```\n\nor the value you think convenient.","commit_id":"37207e38e08fc4f185a8c6288dce2937b5000b72"}]}