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

Calendar doesn't respect timezone for all-day event reminders #27987

Closed
dgsiegel opened this issue Oct 15, 2020 · 9 comments
Closed

Calendar doesn't respect timezone for all-day event reminders #27987

dgsiegel opened this issue Oct 15, 2020 · 9 comments
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap 25-feedback bug feature: caldav Related to CalDAV internals feature: dav

Comments

@dgsiegel
Copy link

Suppose I add an all-day event in the future and add an email reminder to e.g. 1 day before at 9am (which is the default). Being in CEST, I'd expect the reminder at 9am the day before, but it is actually sent 2 hours later (the difference from UTC to CEST).

BEGIN:VCALENDAR
PRODID:-//IDN nextcloud.com//Calendar app 2.0.4//EN
CALSCALE:GREGORIAN
VERSION:2.0
BEGIN:VEVENT
CREATED:20201015T180355Z
DTSTAMP:20201015T190502Z
LAST-MODIFIED:20201015T190502Z
SEQUENCE:6
UID:c8097e7a-0eeb-4b69-abc7-e5f27754e2df
DTSTART;VALUE=DATE:20201018
DTEND;VALUE=DATE:20201019
SUMMARY:All Day Test
BEGIN:VALARM
ACTION:EMAIL
TRIGGER;RELATED=START:-PT15H
END:VALARM
END:VEVENT
END:VCALENDAR

My guess would be that adding the current timezone to DTSTART;VALUE=DATE:20201018 would fix this issue.

@maddylliieeee
Copy link

maddylliieeee commented Jul 5, 2021

Very same issue here, so it exists.

Tested one Reminder for a task at a specific time. The mail comes as expected.
Tested one Reminder for a task which is all-day. The mail comes 2 hours late.

Nextcloud 21.0.2, Calendar app 2.3.0

@tcitworld tcitworld transferred this issue from nextcloud/calendar Jul 15, 2021
@tcitworld tcitworld added 1. to develop Accepted and waiting to be taken care of bug feature: dav labels Jul 15, 2021
@tcitworld
Copy link
Member

There's probably already an issue for that.

@tcitworld tcitworld added the feature: caldav Related to CalDAV internals label Dec 17, 2021
@Pilzinsel64
Copy link

Can confirm, the same. The notification arrives about two hours later then the event. E.g. if I set up to remind on 5pm I get it on 7pm. The calendar app on my mobile phone with DavX5 synced calendars send me a native notification at 5pm. Only the notification via Nextcloud needs at leat 2h longer.

@szaimen
Copy link
Contributor

szaimen commented Jan 23, 2023

Hi, please update to 24.0.9 or better 25.0.3 and report back if it fixes the issue. Thank you!

My goal is to add a label like e.g. 25-feedback to this ticket of an up-to-date major Nextcloud version where the bug could be reproduced. However this is not going to work without your help. So thanks for all your effort!

If you don't manage to reproduce the issue in time and the issue gets closed but you can reproduce the issue afterwards, feel free to create a new bug report with up-to-date information by following this link: https://github.com/nextcloud/server/issues/new?assignees=&labels=bug%2C0.+Needs+triage&template=BUG_REPORT.yml&title=%5BBug%5D%3A+

@szaimen szaimen added needs info 0. Needs triage Pending check for reproducibility or if it fits our roadmap and removed 1. to develop Accepted and waiting to be taken care of labels Jan 23, 2023
@Pilzinsel64
Copy link

Pilzinsel64 commented Jan 24, 2023

I've tested again with v25.0.3 yesterday and the bug is still there. The reminder was planned for 14:00 and my phone's calender app pushed the reminder at 14:00 but Nextcloud pushed the reminder at 15:00. My timezone on all devices is MEZ.

@ChristophWurst
Copy link
Member

Fixed by #36192

@Luncheon3462
Copy link

I’m just an end user, not a dev but running NC 26.0.3. The time zone drift for all day event reminders still occurs for me. Doesn’t matter if I use the personal calendar that is automatically created or a new calendar that I create. My time zone is set correctly in the nc calendar app settings app. Is this expected still?

@Fernando-Checa
Copy link

Same here.. I'm on v28 and allday events are still off by one hour.
Regular events are ok.

@ChristophWurst
Copy link
Member

#43113

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap 25-feedback bug feature: caldav Related to CalDAV internals feature: dav
Projects
None yet
Development

No branches or pull requests

8 participants