Skip to content

Expose boards as calendars - #1545

Merged
juliusknorr merged 15 commits into
masterfrom
enh/dav-calendars
Sep 15, 2020
Merged

Expose boards as calendars#1545
juliusknorr merged 15 commits into
masterfrom
enh/dav-calendars

Conversation

@juliusknorr

@juliusknorrjuliusknorr commented Feb 16, 2020

Copy link
Copy Markdown
Member

@raimund-schluessler

This comment has been minimized.

@juliusknorr

This comment has been minimized.

Comment threadlib/DAV/Calendar.php
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>

@jakobroehrljakobroehrl left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's working fine :)
But I saw the due hour is +1h in the calendar. A 12:00 due card is in the calendar show as a 13:00 due event

@hitam4450

hitam4450 commented Sep 15, 2020

Copy link
Copy Markdown

Maybe timezone problem ?
Daylight saving time ?

Signed-off-by: Julius Härtl <jus@bitgrid.net>
@juliusknorr

Copy link
Copy Markdown
MemberAuthor

Hm, I didn't have this when testing, but I pushed a commit that should force UTC interpretation for the duedate so that the clients can take care of the timezone. Could you check if that fixes it for you?

@jakobroehrl

Copy link
Copy Markdown
Contributor

Hm, I didn't have this when testing, but I pushed a commit that should force UTC interpretation for the duedate so that the clients can take care of the timezone. Could you check if that fixes it for you?

Now it's working.

@juliusknorr

Copy link
Copy Markdown
MemberAuthor

Let's get this in for the next beta 🚀

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@juliusknorr@raimund-schluessler@hitam4450@jakobroehrl@georgehrke