Bowrna commented on code in PR #38482:
URL: https://github.com/apache/airflow/pull/38482#discussion_r1538666652
##########
airflow/www/templates/airflow/dag.html:
##########
@@ -151,7 +151,8 @@ <h4 class="pull-right js-dataset-triggered"
style="user-select: none;-moz-user-s
{% endif %}
{% if dag_model is defined and dag_model.next_dagrun is defined and
dag_model.schedule_interval != 'Dataset' %}
<p class="label label-default js-tooltip" style="margin-left: 5px"
id="next-run" data-html="true" data-placement="bottom">
- Next Run ID: <time datetime="{{ dag_model.next_dagrun }}">{{
dag_model.next_dagrun }}</time>
+ Next Run ID: <time datetime="{{ dag_model.next_dagrun }}">{{
dag_model.next_dagrun }}
+ {{ dag_model.next_dagrun.tzinfo }}</time>
Review Comment:
@bbovenzi @pierrejeambrun With the fix added in the dag.html, I could see
that time has timezone info (both in text UTC and numerical +00:00
representation ) on loading, but then after loading it goes back to the old
state. Am I missing something to notice?
<img width="1041" alt="Screenshot 2024-03-26 at 12 07 41 PM"
src="https://github.com/apache/airflow/assets/10162465/cc7d4882-c6f6-4c99-8758-68869eec6217";>
--
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]
