Eliminate DateTimeTz and just stick to chrono DateTime with a FixedOffset #129

savanni · 2023-12-27T22:25:47Z

savanni commented

2023-12-27 22:25:47 +00:00

DateTimeTz existed to try to work around storing dates and times while also preserving the timezone. The naive date is supposed to be the date that the person originally recorded the record in, no matter what time zone the person is now in.

However, I think if I record a date with the offset, and manage to re-read that date along with the offset, I can still ask DateTime for the naive_date and get the correct answer.

DateTimeTz existed to try to work around storing dates and times while also preserving the timezone. The naive date is supposed to be the date that the person originally recorded the record in, no matter what time zone the person is now in. However, I think if I record a date with the offset, and manage to re-read that date along with the offset, I can still ask DateTime for the naive_date and get the correct answer.

savanni added 1 commit 2023-12-27 22:25:48 +00:00

c94b7db484 Stop using DateTimeTz

savanni merged commit c94b7db484 into main

2023-12-27 22:25:56 +00:00

savanni deleted branch emseries/streamlined-timestamps

2023-12-27 22:25:56 +00:00

savanni referenced this pull request

2023-12-28 17:46:59 +00:00

Drop DateTimeTz from FitnessTrax #131

Sign in to join this conversation.

No reviewers