Skip to content

Conversation

@jscheffl
Copy link
Contributor

@jscheffl jscheffl commented Oct 4, 2025

While I was prepping PR #56392 I noticed that the Task Display Name is not used (anymore?) in the Grid view. Might be a victim of refactoring... this PR adds the Task Display name back again such that "beautiful" task names as introduced in Airflow 2.10 are back again in Grid view.

Before:
image

After:
image

@jscheffl jscheffl added this to the Airflow 3.1.1 milestone Oct 4, 2025
@jscheffl jscheffl added the type:bug-fix Changelog: Bug Fixes label Oct 4, 2025
@jscheffl jscheffl requested a review from bbovenzi as a code owner October 4, 2025 19:15
@jscheffl jscheffl added the area:UI Related to UI/UX. For Frontend Developers. label Oct 4, 2025
@boring-cyborg boring-cyborg bot added the area:API Airflow's REST/HTTP API label Oct 4, 2025
Copy link
Member

@hussein-awala hussein-awala left a comment

Choose a reason for hiding this comment

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

LGTM

@jscheffl jscheffl added the backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch label Oct 4, 2025
@jscheffl jscheffl force-pushed the bugfix/use-task-display-name-in-grid branch from 114e6fb to c664f9a Compare October 4, 2025 21:34
Copy link
Member

@guan404ming guan404ming left a comment

Choose a reason for hiding this comment

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

+1 for using task_display_name in ui. I think re-run static check could fix the red ci, thanks!

@jscheffl jscheffl merged commit d43b468 into apache:main Oct 5, 2025
111 checks passed
@github-actions
Copy link

github-actions bot commented Oct 5, 2025

Backport failed to create: v3-1-test. View the failure log Run details

Status Branch Result
v3-1-test Commit Link

You can attempt to backport this manually by running:

cherry_picker d43b468 v3-1-test

This should apply the commit to the v3-1-test branch and leave the commit in conflict state marking
the files that need manual conflict resolution.

After you have resolved the conflicts, you can continue the backport process by running:

cherry_picker --continue

jscheffl added a commit to jscheffl/airflow that referenced this pull request Oct 5, 2025
* Use Task Display Name in Grid if existing

* Consider short task ID w/o prefix if no display name

* Add some tests with labels on tasks

* Apply ruff
(cherry picked from commit d43b468)

Co-authored-by: Jens Scheffler <95105677+jscheffl@users.noreply.github.com>
jscheffl added a commit that referenced this pull request Oct 5, 2025
* Use Task Display Name in Grid if existing

* Consider short task ID w/o prefix if no display name

* Add some tests with labels on tasks

* Apply ruff
(cherry picked from commit d43b468)
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 7, 2025
* Use Task Display Name in Grid if existing

* Consider short task ID w/o prefix if no display name

* Add some tests with labels on tasks

* Apply ruff
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 8, 2025
* Use Task Display Name in Grid if existing

* Consider short task ID w/o prefix if no display name

* Add some tests with labels on tasks

* Apply ruff
Copy link
Member

@pierrejeambrun pierrejeambrun left a comment

Choose a reason for hiding this comment

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

Nice, thanks

abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 9, 2025
* Use Task Display Name in Grid if existing

* Consider short task ID w/o prefix if no display name

* Add some tests with labels on tasks

* Apply ruff
jscheffl added a commit to jscheffl/airflow that referenced this pull request Oct 9, 2025
* Use Task Display Name in Grid if existing

* Consider short task ID w/o prefix if no display name

* Add some tests with labels on tasks

* Apply ruff
(cherry picked from commit d43b468)

Co-authored-by: Jens Scheffler <95105677+jscheffl@users.noreply.github.com>
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 10, 2025
* Use Task Display Name in Grid if existing

* Consider short task ID w/o prefix if no display name

* Add some tests with labels on tasks

* Apply ruff
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 11, 2025
* Use Task Display Name in Grid if existing

* Consider short task ID w/o prefix if no display name

* Add some tests with labels on tasks

* Apply ruff
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 12, 2025
* Use Task Display Name in Grid if existing

* Consider short task ID w/o prefix if no display name

* Add some tests with labels on tasks

* Apply ruff
dabla pushed a commit to dabla/airflow that referenced this pull request Oct 12, 2025
* Use Task Display Name in Grid if existing

* Consider short task ID w/o prefix if no display name

* Add some tests with labels on tasks

* Apply ruff
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 14, 2025
* Use Task Display Name in Grid if existing

* Consider short task ID w/o prefix if no display name

* Add some tests with labels on tasks

* Apply ruff
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 15, 2025
* Use Task Display Name in Grid if existing

* Consider short task ID w/o prefix if no display name

* Add some tests with labels on tasks

* Apply ruff
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 17, 2025
* Use Task Display Name in Grid if existing

* Consider short task ID w/o prefix if no display name

* Add some tests with labels on tasks

* Apply ruff
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 19, 2025
* Use Task Display Name in Grid if existing

* Consider short task ID w/o prefix if no display name

* Add some tests with labels on tasks

* Apply ruff
TyrellHaywood pushed a commit to TyrellHaywood/airflow that referenced this pull request Oct 22, 2025
* Use Task Display Name in Grid if existing

* Consider short task ID w/o prefix if no display name

* Add some tests with labels on tasks

* Apply ruff
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:API Airflow's REST/HTTP API area:UI Related to UI/UX. For Frontend Developers. backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch type:bug-fix Changelog: Bug Fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants