Skip to content

Add conf column into Dag Runs page #51270

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Jun 2, 2025
Merged

Add conf column into Dag Runs page #51270

merged 1 commit into from
Jun 2, 2025

Conversation

valentinDruzhinin
Copy link
Contributor

The screenshots:

Знімок екрана 2025-05-31 о 23 04 28 Знімок екрана 2025-05-31 о 23 04 43

closes: #50912

@boring-cyborg boring-cyborg bot added the area:UI Related to UI/UX. For Frontend Developers. label Jun 1, 2025
@jscheffl
Copy link
Contributor

jscheffl commented Jun 1, 2025

Thanks for the addition. I know that in some cases conf can be getting large, this will then clutter the layout. Can you make it similar to #50100 such that a dictionary is rendered and it is collapsible and can be expanded on demand?

@valentinDruzhinin
Copy link
Contributor Author

Thanks for the addition. I know that in some cases conf can be getting large, this will then clutter the layout. Can you make it similar to #50100 such that a dictionary is rendered and it is collapsible and can be expanded on demand?

Thank you, updated :)
Here are the new screenshots:
Знімок екрана 2025-06-01 о 17 40 21
Знімок екрана 2025-06-01 о 17 39 56
Знімок екрана 2025-06-01 о 17 40 03

@valentinDruzhinin valentinDruzhinin force-pushed the fix-50912 branch 2 times, most recently from fdf97ed to 0566ef5 Compare June 1, 2025 23:17
jscheffl
jscheffl approved these changes Jun 2, 2025
View reviewed changes
Copy link
Contributor

@jscheffl jscheffl left a comment

Choose a reason for hiding this comment

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

Thanks for the extension, the rework for translation and JSON field rendering. Looking great!

@jscheffl jscheffl merged commit e64f5b3 into apache:main Jun 2, 2025
43 checks passed
@valentinDruzhinin
Copy link
Contributor Author

Thanks for the extension, the rework for translation and JSON field rendering. Looking great!

Thank you :)

@pierrejeambrun
Copy link
Member

I seems there's no backport PR, new feature + depends on i18n => marked for 3.1.0.

Can definitely be backported to next patch release if we don't want to wait.

sanederchik pushed a commit to sanederchik/airflow that referenced this pull request Jun 7, 2025
@valentinDruzhinin @sanederchik
Add conf column into Dag Runs page (apache#51270)
b23682f 
Co-authored-by: Valentyn Druzhynin <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jscheffl jscheffl jscheffl approved these changes

@bbovenzi bbovenzi Awaiting requested review from bbovenzi bbovenzi is a code owner

@pierrejeambrun pierrejeambrun Awaiting requested review from pierrejeambrun pierrejeambrun is a code owner

@ryanahamilton ryanahamilton Awaiting requested review from ryanahamilton ryanahamilton is a code owner

@shubhamraj-git shubhamraj-git Awaiting requested review from shubhamraj-git shubhamraj-git is a code owner

Assignees
No one assigned
Labels
area:UI Related to UI/UX. For Frontend Developers.
Projects
None yet
Milestone
Airflow 3.1.0
Development

Successfully merging this pull request may close these issues.
3 participants
@valentinDruzhinin @jscheffl @pierrejeambrun