bbovenzi commented on a change in pull request #22123:
URL: https://github.com/apache/airflow/pull/22123#discussion_r837938241
##########
File path: airflow/www/views.py
##########
@@ -2554,6 +2552,20 @@ def grid(self, dag_id, session=None):
'dag_runs': encoded_runs,
}
+ dag_details_url = ''
+ tasks_url = ''
+ # This is a workround to use try/except if url_for() throws an error
Review comment:
During tests `url_for` was throwing an error, so I moved the logic into
`views.py` but I'm all ears on a better way to handle this.
In the future, it would be nice to have a sort of `url_for(RestAPI)`
function.
--
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]