search

sipb / hydrant

MIT semester course planning app
https://hydrant.mit.edu/
MIT License
27 stars 18 forks source link

ics export should be timezone-aware #26

Closed cjquines closed 11 months ago

cjquines commented 1 year ago

@dolphingarlic

cjquines commented 1 year ago

gonna bump this, it's gonna come up soon and we're going to get a bunch of complaints. @SuperC03

cjquines commented 1 year ago

i can still repro this; i think the problem is that it exports events in UTC rather than eastern

cjquines commented 1 year ago

we got another email about this so i am escalating again, @psvenk can you assign someone to investigate?

psvenk commented 1 year ago

we got another email about this so i am escalating again, @psvenk can you assign someone to investigate?

I investigated a bit earlier today and couldn't pin down what was going wrong. (The specific scenario from the person who emailed seems like it shouldn't happen if we're exporting with the correct absolute time in UTC, but I haven't tried reproducing on a machine set to a non-ET time zone; I am aware, OTOH, of the DST issue.) I'll look at it some more and/or assign someone in the next few days.