You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/04/07 14:23:53 UTC

[GitHub] [airflow] uranusjr commented on pull request #11652: Support for sorting DAGs in the web UI

uranusjr commented on pull request #11652:
URL: https://github.com/apache/airflow/pull/11652#issuecomment-814957276


   I like this, but a thing that worries me is we’ve recently merged filtering logic for the API, and with this we’ll have two mostly identical sets of logic for sorting DAGs, one in Python (for the API), and one in Jinja2 (for the web UI). Would it be possible to move most of the logic into the views instead? This would make it much easier to potentially refactor the two to use one implementation (see #15072).


-- 
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:
users@infra.apache.org