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

Correct TZ #68

Open
xdadda opened this issue Sep 25, 2016 · 2 comments
Open

Correct TZ #68

xdadda opened this issue Sep 25, 2016 · 2 comments

Comments

@xdadda
Copy link

xdadda commented Sep 25, 2016

Hi guys
I had some serious issue with TZID datetimes (as used by apple icalendar) when the server was in a different timezone vs the browser
unfortunately ical doesn't handle it correctly

this is my current workaround:

  • install moment-timezone.js
  • replace row 131-137 in ical.js with

curr[name] = require('moment-timezone').tz(val.toString(), (parseParams(params).TZID || '')).tz("UTC").toDate()

hope this will help others in my same situation

yyolk added a commit to yyolk/node-ical-improved that referenced this issue Jul 29, 2017
@yyolk
Copy link

yyolk commented Jul 29, 2017

should this actually be

newDate = require('moment-timezone').tz(val.toString(), (parseParams(params).TZID || '')).tz("UTC").toDate(); 

@abrantes01
Copy link

Any news on this ? Can we merge @yyolk solution to this repo or should I use his fork ?

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

No branches or pull requests

3 participants