bbovenzi commented on issue #64847: URL: https://github.com/apache/airflow/issues/64847#issuecomment-4547105109
Looking great! dags list card - The filter is broken, since the dag card won't show the required actions badge without actually filtering the dags list for it. I think we need to fix the issue that we can't tell which dags will have HITL without expliciting filtering for it or waiting to get a 404 on fetching HITL details per dag. That is a separate PR. But I do like that clicking on the badge on the dag card would open the same modal. dag page - Let's move the pending actions button into the header. We should remove the tab navigation in favor of this modal set up. But we will need backward compatibility of any old links by checking the URL route and opening the modal. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
