Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Always use America/Los_Angeles time zone #677

Open
flwyd opened this issue Jul 28, 2023 · 2 comments
Open

Always use America/Los_Angeles time zone #677

flwyd opened this issue Jul 28, 2023 · 2 comments
Labels

Comments

@flwyd
Copy link

flwyd commented Jul 28, 2023

Doing the WESL training there are questions like "What rangers responded to an SV incident at 04:30@D on 7/31@20:50". I'm using the IMS training server from Mountain Daylight Time (America/Denver), and that incident shows in the UI at 21:50.

On the assumption that all IMS incidents happen in Black Rock City, I'm of the opinion that times should always be displayed in Pacific Daylight Time, not whatever time zone the user's browser happens to think it's in.

Besides the training use case, this could be relevant if we continue the remote-operator setup that (I think) we used some in 2022. If someone's operating from Sydney, both computer-generated and free-text dates and times should use the same Nevada time reference, and not mix in New South Wales times.

@wsanchez
Copy link
Member

wsanchez commented Oct 26, 2023

OK this is reporting behavior in the web client built into the server, not this one, but we'll keep this to verify that the problem doesn't carry forward.

I'm pretty sure it does, because the browser should be displaying date-times in the local time zone of the user…

@wsanchez wsanchez added the Bug label Oct 26, 2023
@srabraham
Copy link
Member

I made a relevant update in the legacy IMS web client, which at least removed ambiguity about time zones in the UI (burningmantech/ranger-ims-server#1412 (comment)

I agree with @flwyd that it'd be sensible and less confusing to only show America/Los_Angeles times in the web clients. In the backend, maybe an Event could be associated with a tz string, and that would be fed to the web clients to help them render datetimes?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants