Skip to content

Conversation

@edwardwang888
Copy link
Contributor

I made two changes to the UI datetime format:

  1. Display the date in the current timestamp. It was removed during the Airflow 2.0 UI overhaul, but I think it is still useful to have it displayed, because:
    • we now display Next Run with the date
    • it is helpful when viewing timestamps in the logs and trying to figure out how long ago something executed
  2. Make sure the timezone is still displayed in the logs after changing the default timezone

  1. Refresh the page to get initial behavior. The default timezone is UTC.
  • Before:
    • Timezone is displayed in the logs before changing default timezone.
      image
      image
  • After:
    • Current timestamp format is updated.
      image
      image
  1. Change default timezone to PDT.
  • Before:
    • Timezone disappears from logs.
      image
      image
  • After:
    • Timezone is still displayed in logs.
      image
      image
  1. Change default timezone back to UTC.
  • Before:
    • Timezone missing from logs.
      image
      image
  • After:
    • Timezone is still displayed in logs.
      image
      image

^ Add meaningful description above

Read the Pull Request Guidelines for more information.
In case of fundamental code change, Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in UPDATING.md.

The Airflow 2.0 UI overhaul removed the date from being displayed in the current timestamp. However, it is still helpful to show the date, because:

* we now display `Next Run` with the date
* it is helpful when viewing timestamps in the logs and trying to figure out how long ago something executed

For consistency, the format has been changed to match the format that `Next Run` uses.
Without this change, the timezone is no longer displayed after changing the default timezone.
@boring-cyborg boring-cyborg bot added area:UI Related to UI/UX. For Frontend Developers. area:webserver Webserver related Issues labels Oct 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:UI Related to UI/UX. For Frontend Developers. area:webserver Webserver related Issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant