The GitHub Actions job "Tests (AMD)" on airflow.git/backport-322-66198 has 
succeeded.
Run started by GitHub user vatsrahul1001 (triggered by vatsrahul1001).

Head commit for run:
c340993170c9df63e5b315dc2c6015158d02c746 / internetcoffeephone 
<[email protected]>
UI: Fix "Mark state as..." buttons grayed out when task/DAGRun already in 
target state (#66198)

* UI: Fix "Mark state as..." buttons grayed out when task/DAGRun already in 
target state

Menu items for marking a task instance, task group, or DAG run as
success/failed were disabled whenever the item's current state matched
the target state. This blocked users from re-applying the same state
(e.g. marking an already-succeeded DAG run as success to also flip all
its tasks). The same bug existed in Airflow 2.x (#36219) and regressed
in the Airflow 3.x UI redesign.

Remove the state-equality guards from:
- the `disabled` prop on Menu.Item
- the `onClick` early-return
- the hotkey `enabled` conditions
- the Tooltip `disabled` condition

Add regression tests for MarkRunAsButton to prevent future recurrence.

Fixes #66197

* Replace unit test with inline comments explaining why state-match is not 
disabled

Per review feedback: comments in the source communicate the intent more
reliably than a separate test file that could be deleted without context.

https://claude.ai/code/session_012oET1NyiNZe44zWbr9GZnn

---------

Co-authored-by: Claude <[email protected]>
(cherry picked from commit 505924feaa4de521c364f24d7562a35d951561f0)

Report URL: https://github.com/apache/airflow/actions/runs/25850292117

With regards,
GitHub Actions via GitBox


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to