Skip to content

[Chore] Added page to debug time zone issue#946

Merged
alexjustesen merged 1 commit intomainfrom
time-zone-debug-page
Nov 29, 2023
Merged

[Chore] Added page to debug time zone issue#946
alexjustesen merged 1 commit intomainfrom
time-zone-debug-page

Conversation

@alexjustesen
Copy link
Owner

Description

This PR introduces a timezone debut page located at /debug/timezone which can be used to help resolve local time issues.

🤔 but how do I know if I have a problem?

  • PHP time zone should always read UTC.
  • App time zone should always read UTC.
  • Settings time zone should reflect the same value as the UI with the correct offset for you time zone vs. UTC.
  • UTC time should be the correct time.
  • Local time should be the correct local time factoring in the time zone setting.

Changelog

Added

  • time zone debug page

Screenshots

image

@alexjustesen alexjustesen added the chore Nothing fancy, just needs to get done label Nov 29, 2023
@alexjustesen alexjustesen self-assigned this Nov 29, 2023
@alexjustesen alexjustesen merged commit c09337e into main Nov 29, 2023
@alexjustesen alexjustesen deleted the time-zone-debug-page branch November 29, 2023 22:35
@alexjustesen alexjustesen mentioned this pull request Nov 29, 2023
@hoorna
Copy link

hoorna commented Nov 29, 2023

  • Local time should be the correct local time factoring in the time zone setting.

What do you mean with "in the time zone setting". Are you referring here to the TZ environment variable?

@alexjustesen
Copy link
Owner Author

  • Local time should be the correct local time factoring in the time zone setting.

What do you mean with "in the time zone setting". Are you referring here to the TZ environment variable?

In the application under Settings and General.

@hoorna
Copy link

hoorna commented Nov 29, 2023

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore Nothing fancy, just needs to get done

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants