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

Broker: add timezone to CANCEL messages #191

Merged
merged 1 commit into from
Jan 5, 2019
Merged

Conversation

kesselb
Copy link
Contributor

@kesselb kesselb commented Dec 15, 2018

Ref nextcloud/server#12946

If DTSTART and/or DTEND and other times are specified with a TZID, some clients
may fail reading the generated iCal data, since the VTIMEZONE definition is
missing.

I dont use Outlook but code make sense 🙈 Backport these changes to stable15?

Copy link
Member

@georgehrke georgehrke left a comment

Choose a reason for hiding this comment

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

The patch make sense and should be safe to back port imho 👍

@georgehrke
Copy link
Member

/backport to stable15

@georgehrke
Copy link
Member

/backport to stable14

@backportbot-nextcloud
Copy link

backport to stable15 in #196

@backportbot-nextcloud
Copy link

backport to stable14 in #197

If DTSTART and/or DTEND and other times are specified with a TZID, some clients
may fail reading the generated iCal data, since the VTIMEZONE definition is
missing.

Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
@kesselb kesselb merged commit 9a5103d into master Jan 5, 2019
@kesselb kesselb deleted the add-cancel-message branch January 5, 2019 16:04
hdijkema added a commit to hdijkema/server that referenced this pull request Jan 14, 2019
Sign up for free to 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.

4 participants