Add pendingActions onto ui/dags#55041
Conversation
|
Need to do some more "Required Actions" -> "Needs Review" renaming but in another PR |
jscheffl
left a comment
There was a problem hiding this comment.
Filter looks good in general but ... naming is hard. I am not 100% OK with "Needs Review". Functional and Code is perfect!
|
If you plan to merge it before the 3.1 release, please consider my comment in this PR: #55119 (comment) |
sure :) |
b1421c8 to
f695cba
Compare
ddf2906 to
d9597b7
Compare
pierrejeambrun
left a comment
There was a problem hiding this comment.
Just one nit, but looks good. Thanks.
Thanks for the reminder. I'll try to reduce the i18n surface area of this PR and keep any renaming discussion on a follow up branch. |
Related: #54961
Add hitl that need reviews as
pending_actionson ui/dags. Show "Needs Review" as a badge on the dags list. Allow a user to filter the dags list to only show dags that need review. Dashboard link now redirects to the dags list filtered.^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named
{pr_number}.significant.rstor{issue_number}.significant.rst, in airflow-core/newsfragments.