mik-laj commented on a change in pull request #9503:
URL: https://github.com/apache/airflow/pull/9503#discussion_r447755902



##########
File path: airflow/api_connexion/endpoints/dag_run_endpoint.py
##########
@@ -69,24 +61,24 @@ def get_dag_runs(session, dag_id, start_date_gte=None, 
start_date_lte=None,
 
     # filter start date
     if start_date_gte:
-        query = query.filter(DagRun.start_date >= start_date_gte)
+        query = query.filter(DagRun.start_date >= 
timezone.parse(start_date_gte))

Review comment:
       I've thought about it and think it's worth keeping the `format_datetime` 
functions to convert parameter types. Then all view function will not have to 
convert types. We will have logic separations.




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

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


Reply via email to