)]}'
{"releasenotes/notes/fix-actions-table-display-choices-a3b2c1d4e5f6.yaml":[{"robot_id":"zuul","robot_run_id":"3af617bca6db443f8439c692cb1d88b7","url":"https://zuul.teim.app/t/main/buildset/3af617bca6db443f8439c692cb1d88b7","author":{"_account_id":28006,"name":"teim-ci","display_name":"teim-ci","email":"ci@seanmooney.info","username":"ci-sean-mooney","status":"this is a third-party ci account run by sean-k-mooney on irc\nhosted at zuul.teim.app"},"tag":"autogenerated:zuul:automatic-ci","change_message_id":"450fa5719c715ed3b02198e0777bdb74f2c08975","patch_set":15,"id":"1d0349e3_dbdbc1c9","line":1,"updated":"2025-11-13 09:49:26.000000000","message":"Release note could be more specific - (Severity: warning, Confidence: 0.8)","commit_id":"56a01a93d3980eb368afb63272724404a140a999"},{"robot_id":"zuul","robot_run_id":"55fe7be450a34847ab316ef62c5783fb","url":"https://zuul.teim.app/t/main/buildset/55fe7be450a34847ab316ef62c5783fb","author":{"_account_id":28006,"name":"teim-ci","display_name":"teim-ci","email":"ci@seanmooney.info","username":"ci-sean-mooney","status":"this is a third-party ci account run by sean-k-mooney on irc\nhosted at zuul.teim.app"},"tag":"autogenerated:zuul:automatic-ci","change_message_id":"dd808efd61ba26be7fec78f9a46d2e85edd7ade8","patch_set":17,"id":"b199d6d9_d3884c85","line":1,"updated":"2026-04-28 10:22:49.000000000","message":"The release note filename fix-actions-table-display-choices-a3b2c1d4e5f6.yaml mentions only \u0027actions table\u0027 but the fix covers action_plans and audits tables as well.\n\n**Severity**: SUGGESTION | **Confidence**: 0.7\n\n**Benefit**: A more descriptive filename would help release note readers quickly identify the full scope of the fix.\n\n**Recommendation**:\nConsider a more general filename like fix-table-state-display-choices-\u003chash\u003e.yaml in a follow-up patch. This is cosmetic and does not affect functionality; reno works fine with the current name.","commit_id":"bdd0c8018be485bc0b3ea31ae90cfd6e4206d888"}],"watcher_dashboard/content/action_plans/tables.py":[{"robot_id":"zuul","robot_run_id":"55fe7be450a34847ab316ef62c5783fb","url":"https://zuul.teim.app/t/main/buildset/55fe7be450a34847ab316ef62c5783fb","author":{"_account_id":28006,"name":"teim-ci","display_name":"teim-ci","email":"ci@seanmooney.info","username":"ci-sean-mooney","status":"this is a third-party ci account run by sean-k-mooney on irc\nhosted at zuul.teim.app"},"tag":"autogenerated:zuul:automatic-ci","change_message_id":"dd808efd61ba26be7fec78f9a46d2e85edd7ade8","patch_set":17,"id":"1cf668b4_d0bff5b5","line":166,"updated":"2026-04-28 10:22:49.000000000","message":"The commit message body could more explicitly note that status\u003dTrue was also removed alongside status_choices, since this changes row CSS styling behaviour.\n\n**Severity**: SUGGESTION | **Confidence**: 0.8\n\n**Benefit**: Reviewers and future readers would understand the full scope of the visual change: not only are display labels now using the correct API, but rows will no longer have status_up/status_down CSS classes applied.\n\n**Recommendation**:\nConsider adding a sentence to the commit body such as: \u0027The status\u003dTrue parameter was also removed from the affected columns since it was enabling row-level status colouring (green/red CSS classes) that is not appropriate for these multi-state columns.\u0027 This is optional since the current message already describes the change well.","commit_id":"bdd0c8018be485bc0b3ea31ae90cfd6e4206d888"}],"watcher_dashboard/content/actions/tables.py":[{"robot_id":"zuul","robot_run_id":"3af617bca6db443f8439c692cb1d88b7","url":"https://zuul.teim.app/t/main/buildset/3af617bca6db443f8439c692cb1d88b7","author":{"_account_id":28006,"name":"teim-ci","display_name":"teim-ci","email":"ci@seanmooney.info","username":"ci-sean-mooney","status":"this is a third-party ci account run by sean-k-mooney on irc\nhosted at zuul.teim.app"},"tag":"autogenerated:zuul:automatic-ci","change_message_id":"450fa5719c715ed3b02198e0777bdb74f2c08975","patch_set":15,"id":"09ed00e7_ab494df6","line":32,"updated":"2025-11-13 09:49:26.000000000","message":"Translation context not fixed in ACTION_STATE_DISPLAY_CHOICES - (Severity: critical, Confidence: 1.0) | Risk: Usability/Translation | Priority: Fix Now | Why This Matters: The commit message explicitly states it fixes translation context that \"refers to \u0027Power state of an Instance\u0027\" but this was never corrected. The ACTION_STATE_DISPLAY_CHOICES still uses incorrect context \"Power state of an Instance\" instead of \"State of an action\" which will confuse translators and users. | Recommendation: Update ACTION_STATE_DISPLAY_CHOICES to use proper context like pgettext_lazy(\"State of an action\", \"No State\") instead of pgettext_lazy(\"Power state of an Instance\", \"No State\")","commit_id":"56a01a93d3980eb368afb63272724404a140a999"},{"robot_id":"zuul","robot_run_id":"a775e3e4dee94e82831f599d5fccdfa9","url":"https://zuul.teim.app/t/main/buildset/a775e3e4dee94e82831f599d5fccdfa9","author":{"_account_id":28006,"name":"teim-ci","display_name":"teim-ci","email":"ci@seanmooney.info","username":"ci-sean-mooney","status":"this is a third-party ci account run by sean-k-mooney on irc\nhosted at zuul.teim.app"},"tag":"autogenerated:zuul:automatic-ci","change_message_id":"4e88211489d73817ecec002a6f3dee333324876f","patch_set":16,"id":"1fcbe8ac_c70370d2","line":69,"updated":"2025-12-01 00:59:48.000000000","message":"Inconsistent line continuation in actions/tables.py at line 69\n\n**Severity**: SUGGESTION | **Confidence**: 0.8\n\n**Benefit**: Improves code readability and maintains consistency with OpenStack style\n\n**Recommendation**:\nFix line continuation formatting to align with project style. The line should be properly wrapped at 79 characters with consistent indentation.","commit_id":"85955966e2d4035db8dbad2080fe45dc845fa317"},{"robot_id":"zuul","robot_run_id":"55fe7be450a34847ab316ef62c5783fb","url":"https://zuul.teim.app/t/main/buildset/55fe7be450a34847ab316ef62c5783fb","author":{"_account_id":28006,"name":"teim-ci","display_name":"teim-ci","email":"ci@seanmooney.info","username":"ci-sean-mooney","status":"this is a third-party ci account run by sean-k-mooney on irc\nhosted at zuul.teim.app"},"tag":"autogenerated:zuul:automatic-ci","change_message_id":"dd808efd61ba26be7fec78f9a46d2e85edd7ade8","patch_set":17,"id":"7b1c441d_c82fadcd","line":35,"updated":"2026-04-28 10:22:49.000000000","message":"ACTION_STATE_DISPLAY_CHOICES in actions/tables.py uses incorrect pgettext_lazy context string \"Power state of an Instance\" instead of an action-specific context like \"State of an action\".\n\n**Severity**: WARNING | **Confidence**: 0.9\n\n**Impact**: Translators see misleading context for these display labels, which could result in incorrect translations. The context string is copy-pasted from an unrelated Horizon module (instance power states).\n\n**Suggestion**:\nReplace all 8 occurrences of pgettext_lazy(\"Power state of an Instance\", ...) with pgettext_lazy(\"State of an action\", ...) to match the convention used in action_plans/tables.py and audits/tables.py. This is a pre-existing issue not introduced by this patch, but fixing it in the same patch would be consistent since the code is already being touched.","commit_id":"bdd0c8018be485bc0b3ea31ae90cfd6e4206d888"}]}
