pierrejeambrun commented on code in PR #52437:
URL: https://github.com/apache/airflow/pull/52437#discussion_r2190165929


##########
airflow-core/src/airflow/api_fastapi/core_api/routes/public/dag_run.py:
##########
@@ -334,6 +336,7 @@ def get_dag_runs(
     readable_dag_runs_filter: ReadableDagRunsFilterDep,
     session: SessionDep,
     dag_bag: DagBagDep,
+    run_id_pattern: Annotated[_SearchParam, 
Depends(search_param_factory(DagRun.run_id, "run_id"))],

Review Comment:
   ```suggestion
       run_id_pattern: Annotated[_SearchParam, 
Depends(search_param_factory(DagRun.run_id, "run_id_pattern"))],
   ```
   
   (All search params end up with the `pattern` suffix, this will be the name 
of the query parameter)



##########
airflow-core/src/airflow/ui/public/i18n/locales/en/dags.json:
##########
@@ -13,7 +13,8 @@
       "active": "Active",
       "all": "All",
       "paused": "Paused"
-    }
+    },
+    "runIdPatternFilter": "Add run_id filter"

Review Comment:
   Update the translation "Search Runs"



##########
airflow-core/src/airflow/api_fastapi/core_api/openapi/v2-rest-api-generated.yaml:
##########
@@ -2041,6 +2041,18 @@ paths:
           type: string
           default: id
           title: Order By
+      - name: run_id

Review Comment:
   Just realized that this should be `run_id_pattern` cf comment above and 
other search params.



-- 
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.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to