necodeIT / lb_planner_plugin

Moodle Plugin for the LB Planner app
Other
1 stars 0 forks source link

pass along timezone for timedates (mandatory!) #28

Open RiedleroD opened 1 month ago

RiedleroD commented 1 month ago

ALL timedates should have their timezone attached! This is non-negotiable if we want this to work at all outside CET/CEST. I'm not even sure if we're not assuming GMT instead in some places. tbqh the time-related code is a mess and needs to be rewritten. We can't rely on only passing implicitly UTC-based timedates, especially in countries that have summertime/wintertime splits.

I suggest:

todos:

mcquenji commented 1 month ago

or we only store UTC timestamps which can be converted client side to the correct timezone

RiedleroD commented 1 month ago

I literally already explained why we can't do that

mcquenji commented 1 month ago

i dont get it

RiedleroD commented 1 month ago

I'll explain it in tomorrow's meeting. tldr time is fucked and we're the condom