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

outlook live calendar showing different dates #154

Closed
govardhan5 opened this issue Apr 15, 2020 · 16 comments
Closed

outlook live calendar showing different dates #154

govardhan5 opened this issue Apr 15, 2020 · 16 comments
Assignees
Labels
bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed

Comments

@govardhan5
Copy link

It seems like the startdt and enddt provided by us in query param is not getting triggered instead it is taking the current date.

Below is the the event I tried to create.

https://outlook.live.com/calendar/0/deeplink/compose?startdt=20200812T000000&enddt=20200813T120000&subject=Hello%20World&body=Hello%20World.

@AnandChowdhary
Copy link
Owner

Thanks for opening this issue, @govardhan5. Perhaps Outlook changed some of the parameter names when I updated the UI, I'll have a closer look.

@AnandChowdhary AnandChowdhary self-assigned this Apr 15, 2020
@AnandChowdhary AnandChowdhary added the bug Something isn't working label Apr 15, 2020
@govardhan5
Copy link
Author

Thanks for the response @AnandChowdhary . Can you please let me know approximate estimation date to fix this issue if possible?

@AnandChowdhary
Copy link
Owner

Honestly, my plate is a little full at the moment. I might have some time this weekend to look into this, otherwise I'll make sure I get into it next week.

@govardhan5
Copy link
Author

Thanks for the response @AnandChowdhary

@webaware
Copy link

The date/time format has changed.

It now no longer accepts YYYYMMDD\THHmmss,
must now be YYYY-MM-DD\THH:mm:ss

Like so:

https://outlook.live.com/calendar/0/deeplink/compose?startdt=2020-08-12T00:00:00&enddt=2020-08-13T12:00:00&subject=Hello%20World&body=Hello%20World

@AnandChowdhary
Copy link
Owner

That's right, thanks! We're discussing this in InteractionDesignFoundation/add-event-to-calendar-docs#10 and I'll push an update to this repo soon.

@webaware
Copy link

Ah, cool. I did look at that project but somehow missed the PR / issue. Good to see the docs will get the update too! 👍

@paulboocock
Copy link
Contributor

Hi @AnandChowdhary I see that InteractionDesignFoundation/add-event-to-calendar-docs#10 is merged. Any rough ETA on when it might be fixed in this repo?

@AnandChowdhary AnandChowdhary added good first issue Good for newcomers help wanted Extra attention is needed labels Jun 3, 2020
@AnandChowdhary
Copy link
Owner

Feel free to pick this up! I'll need more time to get to it; though it's trivial, my hands are pretty full these days.

@suit0r
Copy link

suit0r commented Aug 27, 2020

an updates on this? the package works great otherwise, but the outlook/office365 links don't receive correct dates, so it defaults to today. i'm passing it an .toISOstring(), but i tried a bunch of things, and nothing worked. google works fine, with ISO.
this is the ics in comparison:

VERSION:2.0
BEGIN:VEVENT
URL:MYURL.com
DTSTART:20200830T160000Z
DTEND:20200830T173000Z
SUMMARY:My meeting
DESCRIPTION:Description here
LOCATION:Some location
END:VEVENT
END:VCALENDAR

@paulboocock
Copy link
Contributor

I've been meaning to submit a PR for this but if you want a quick fix in your application, I've been doing this:

const outlook = calendar.outlook(event);
console.log('Outlook: ' + outlook.replace(/(\d{4})(\d{2})(\d{2})T(\d{2})(\d{2})(\d{2})/g, '$1-$2-$3T$4:$5:$6')); 

@suit0r
Copy link

suit0r commented Aug 28, 2020

thanks a lot. works on both outlook and office calendars.

@AnandChowdhary
Copy link
Owner

I've been meaning to submit a PR for this but if you want a quick fix in your application, I've been doing this:


const outlook = calendar.outlook(event);

console.log('Outlook: ' + outlook.replace(/(\d{4})(\d{2})(\d{2})T(\d{2})(\d{2})(\d{2})/g, '$1-$2-$3T$4:$5:$6')); 

Would love a PR!

@AnandChowdhary
Copy link
Owner

The replacement has been added in v1.3.1, thanks to @paulboocock!

@jonwhuang
Copy link

jonwhuang commented Sep 9, 2020

The outlook/office365 fix doesn't seem to take timezone into account.

If I pass in 20200924T170000Z to 20200924T180000Z for start/end times, it shows different values for Google vs Outlook.

Google converts the times to the user's timezone while outlook seems to always use 5-6PM.

@frysztak
Copy link

frysztak commented Mar 2, 2021

I'm still experiencing this issue at version 2.0.11. Instead of YYYY-MM-DD, YYYYMMDD format is used

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

7 participants