ryanahamilton commented on a change in pull request #16129:
URL: https://github.com/apache/airflow/pull/16129#discussion_r641736594
##########
File path: airflow/www/static/js/task.js
##########
@@ -17,11 +17,18 @@
* under the License.
*/
-/* global document */
-import { formatDateTime } from './datetime_utils';
+/* global document, moment */
-// reformat execution date to be more human-readable
+// reformat task details to be more human-readable
document.addEventListener('DOMContentLoaded', () => {
- const date = document.getElementById('ti_execution_date');
- date.innerHTML = formatDateTime(date.innerHTML);
+ document.querySelectorAll('.ti-attr').forEach((attr) => {
Review comment:
Sorry I didn't catch this last time, but should we change this one as
well?
```suggestion
document.querySelectorAll('.js-ti-attr').forEach((attr) => {
```
--
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]
