Skip to content

UI clear task checkboxes#68029

Merged
choo121600 merged 2 commits into
apache:mainfrom
suhaas-vaddadi:ui-clear-task-checkboxes
Jun 6, 2026
Merged

UI clear task checkboxes#68029
choo121600 merged 2 commits into
apache:mainfrom
suhaas-vaddadi:ui-clear-task-checkboxes

Conversation

@suhaas-vaddadi

@suhaas-vaddadi suhaas-vaddadi commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)

Generated-by: Claude Code following the guidelines

Before

clear-task-instance-before
  • Selecting a task + downstream/past/future resolved full set of tasks and cleared everything
  • To skip subset of downstream tasks, you had to clear it and retrigger manually

After

clear-task-instance-after
  • Every affected task instance renders a checkbox, ticking by default
  • Per-run granularity with past/future expansions you can keep task X in run 1 while dropping it from run 2

Effect on API

  • API calls remain the exact same with no exclusions and lets backend handle set expansion
  • With exclusions, UI sends the set as explicit task_ids with expansion flags off

File by File

ActionAccordion/columns.tsx — Adds taskInstanceKey (dag_run_id:task_id:map_index) and a RowSelection type; getColumns now prepends an optional leading checkbox column when selection is passed.

ActionAccordion/ActionAccordion.tsx — Threads the optional selection prop down to the table columns. No change when omitted.

Clear/TaskInstance/ClearTaskInstanceDialog.tsx — Tracks excludedKeys state and wires checkboxes in. On confirm with exclusions, sends the kept instances explicitly, grouped per run with expansion flags off; otherwise unchanged.

Clear/TaskInstance/ClearTaskInstanceConfirmationDialog.tsx — Accepts optional explicit taskIds; when present, the dry-run preview sends those ids with upstream/downstream off so it matches the real clear.


  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst, in airflow-core/newsfragments. You can add this file in a follow-up commit after the PR is created so you know the PR number.

@boring-cyborg boring-cyborg Bot added the area:UI Related to UI/UX. For Frontend Developers. label Jun 4, 2026
@dheerajturaga

Copy link
Copy Markdown
Member

@suhaas-vaddadi, ci is failing. Take a look?

@dheerajturaga

Copy link
Copy Markdown
Member

@suhaas-vaddadi, Can you also add a way to select and deselect all the tasks in this list?
image

@dheerajturaga

Copy link
Copy Markdown
Member

@suhaas-vaddadi CI failure on main was fixed, can you rebase?

@suhaas-vaddadi suhaas-vaddadi force-pushed the ui-clear-task-checkboxes branch from 0bf7f92 to 2053a55 Compare June 5, 2026 06:23
@suhaas-vaddadi suhaas-vaddadi reopened this Jun 5, 2026

@dheerajturaga dheerajturaga left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great! this works, thanks for contributing!

Another pair of eyes on this would be great!

@choo121600 choo121600 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice feature!
LGTM :)

@choo121600 choo121600 merged commit 31fcbf6 into apache:main Jun 6, 2026
110 of 121 checks passed
@suhaas-vaddadi suhaas-vaddadi deleted the ui-clear-task-checkboxes branch June 8, 2026 17:25
@vatsrahul1001 vatsrahul1001 added this to the Airflow 3.3.0 milestone Jun 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:UI Related to UI/UX. For Frontend Developers.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants