Page MenuHomePhabricator

[dagit] Prefer user timezone on Schedule ticks

Authored by dish on Jan 7 2021, 9:25 PM.



Show the user's timezone for schedule ticks, with the schedule-specific execution timezone shown in the tooltip. This is the reverse of the current behavior.

Based on discussion today.

I also changed timestampToString to use the browser timezone when "Automatic" is set, so that a timezone is at least rendered when included in the format string. This is intended to avoid confusion about execution timezones if the user is using "Automatic" timezone.

Test Plan

View Schedule, verify reversal described above. Verify also that CST appears even though I'm on an "Automatic" setting.

Diff Detail

R1 dagster
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.