Re: [PR] remove display of param from the UI [airflow]

2024-04-02 Thread via GitHub
hussein-awala commented on PR #38660: URL: https://github.com/apache/airflow/pull/38660#issuecomment-2032783818 > > Ah - so does it need to be displayed ? > > > > Yes, but it's a bit broken. So I can work on a real fix for this Waiting for your PR to understand why displ

Re: [PR] remove display of param from the UI [airflow]

2024-04-02 Thread via GitHub
potiuk commented on PR #38660: URL: https://github.com/apache/airflow/pull/38660#issuecomment-2032653376 :eyes: -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscr

Re: [PR] remove display of param from the UI [airflow]

2024-04-02 Thread via GitHub
bbovenzi commented on PR #38660: URL: https://github.com/apache/airflow/pull/38660#issuecomment-2032640007 > Ah - so does it need to be displayed ? Yes, but it's a bit broken. So I can work on a real fix for this -- This is an automated message from the Apache Git Service. To respon

Re: [PR] remove display of param from the UI [airflow]

2024-04-02 Thread via GitHub
potiuk commented on PR #38660: URL: https://github.com/apache/airflow/pull/38660#issuecomment-2032634406 Ah - so does it need to be displayed ? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to th

Re: [PR] remove display of param from the UI [airflow]

2024-04-02 Thread via GitHub
bbovenzi commented on PR #38660: URL: https://github.com/apache/airflow/pull/38660#issuecomment-2032626711 `root` allows a user to filter the grid+graph by only showing what's upstream/downstream of a task. The UX needs to be improved, but it is still used! -- This is an automated messag

Re: [PR] remove display of param from the UI [airflow]

2024-04-01 Thread via GitHub
uranusjr merged PR #38660: URL: https://github.com/apache/airflow/pull/38660 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@airflow

[PR] remove display of param from the UI [airflow]

2024-04-01 Thread via GitHub
hussein-awala opened a new pull request, #38660: URL: https://github.com/apache/airflow/pull/38660 Displaying the root param in the UI was added in 2015 for some reason, but after checking the current usage, I believe it is completely useless: http://localhost:28080/dags/dag_1/calenda