pierrejeambrun commented on code in PR #43783:
URL: https://github.com/apache/airflow/pull/43783#discussion_r1834503100
##########
airflow/api_fastapi/common/parameters.py:
##########
@@ -406,6 +407,37 @@ def depends(self, dag_id: str | None = None) ->
_DagIdFilter:
return self.set_value(dag_id)
+class _UriPatternSearch(_SearchParam):
+ """Search on uri."""
+
+ def __init__(self, skip_none: bool = True) -> None:
+ super().__init__(AssetModel.uri, skip_none)
+
+ def depends(self, uri_pattern: str | None = None) -> _UriPatternSearch:
Review Comment:
This has changed between flask and fast api.
Fast API natively handles list for query params, and the folmat is
`exploded=True`, i.e `?attr=1&attr=2`.
This is what we are using now, cf other endpoints and test for examples.
--
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]
