search

wuvt / trackman

WUVT's track logging web-app
GNU Affero General Public License v3.0
4 stars 3 forks source link

BMI report assumes input dates are UTC, but outputs in localtime #72

Open echarlie opened 5 years ago

echarlie commented 5 years ago

It is unclear if BMI reports should be in localtime or not, but we definitely should not assume input is in UTC when outputing as localtime, as it can be unclear to the end-user what is happening. We should either accept a full timespec for input, assume input is in localtime, or provide an input timezone list from which the user can select the desired input/output timezone.

mutantmonkey commented 4 years ago

Let's figure out the right approach here. Should we just accept input in local time? I don't think having the user manually specify an input timezone is the way to go, but I wouldn't be opposed to displaying the server timezone on that page.

echarlie commented 4 years ago

Saying "please input dates in America/New_York" or whatever the guessed user timezone is would probably be fine.