kaxil commented on a change in pull request #13922:
URL: https://github.com/apache/airflow/pull/13922#discussion_r567030007
##########
File path: airflow/www/templates/airflow/dags.html
##########
@@ -167,15 +167,15 @@ <h2>DAGs</h2>
<td class="text-center">
<div class="btn-group">
{% if dag %}
- <a href="{{ url_for('Airflow.trigger',
dag_id=dag.dag_id) }}" title="Trigger DAG" aria-label="Trigger DAG" class="btn
btn-sm btn-default btn-icon-only">
- <span class="material-icons"
aria-hidden="true">play_arrow</span>
- </a>
- <a href="{{ url_for('Airflow.refresh',
dag_id=dag.dag_id) }}" onclick="postAsForm(this.href); return false"
title="Refresh DAG" aria-label="Refresh DAG" class="btn btn-sm btn-default
btn-icon-only">
- <span class="material-icons"
aria-hidden="true">refresh</span>
- </a>
+ <a href="{{ url_for('Airflow.trigger',
dag_id=dag.dag_id) }}" title="Trigger DAG" aria-label="Trigger DAG" class="btn
btn-sm btn-default btn-icon-only {{ ' disabled' if not dag.can_trigger else ''
}}">
+ <span class="material-icons"
aria-hidden="true">play_arrow</span>
+ </a>
+ <a href="{{ url_for('Airflow.refresh',
dag_id=dag.dag_id) }}" onclick="postAsForm(this.href); return false"
title="Refresh DAG" aria-label="Refresh DAG" class="btn btn-sm btn-default
btn-icon-only {{ ' disabled' if not dag.can_edit else '' }}">
+ <span class="material-icons"
aria-hidden="true">refresh</span>
+ </a>
{% endif %}
{# Use dag_id instead of dag.dag_id, because the DAG might
not exist in the webserver's DagBag #}
- <a href="{{ url_for('Airflow.delete', dag_id=dag.dag_id)
}}" onclick="return confirmDeleteDag(this, '{{ dag.dag_id }}')"
title="Delete DAG" aria-label="Delete DAG" class="btn btn-sm btn-default
btn-icon-only">
+ <a href="{{ url_for('Airflow.delete', dag_id=dag.dag_id)
}}" onclick="return confirmDeleteDag(this, '{{ dag.dag_id }}')"
title="Delete DAG" aria-label="Delete DAG" class="btn btn-sm btn-default
btn-icon-only {{ ' disabled' if not dag.can_delete else '' }}">
Review comment:
```suggestion
<a href="{{ url_for('Airflow.delete', dag_id=dag.dag_id)
}}" onclick="return confirmDeleteDag(this, '{{ dag.dag_id }}')"
title="Delete DAG" aria-label="Delete DAG" class="btn btn-sm btn-default
btn-icon-only {{ ' disabled' if not dag.can_delete }}">
```
WDYT?
cc @ryanahamilton
----------------------------------------------------------------
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:
[email protected]
