Skip to content

enable caldav for webdav subtree public-calendars - #9798

Merged
MorrisJobke merged 1 commit into
masterfrom
bugfix/noid/enable_caldav_for_public_calendars
Jun 11, 2018
Merged

enable caldav for webdav subtree public-calendars#9798
MorrisJobke merged 1 commit into
masterfrom
bugfix/noid/enable_caldav_for_public_calendars

Conversation

@georgehrke

Copy link
Copy Markdown
Member

fixes public calendar sharing

Signed-off-by: Georg Ehrke <developer@georgehrke.com>
@georgehrkegeorgehrke added this to the Nextcloud 13.0.4 milestone Jun 8, 2018
@georgehrke
georgehrke requested a review from rullzerJune 8, 2018 11:15
@rullzer
rullzer requested a review from MorrisJobkeJune 8, 2018 11:15
@codecov

codecovBot commented Jun 8, 2018

Copy link
Copy Markdown

Codecov Report

Merging #9798 into master will decrease coverage by <.01%.
The diff coverage is 100%.

@@ Coverage Diff @@## master #9798 +/- ##
============================================
- Coverage 51.92% 51.92% -0.01% 
Complexity 25768 25768 ============================================
Files 1635 1635 Lines 95426 95426 Branches 1308 1308 ============================================
- Hits 49553 49552 -1 - Misses 45873 45874 +1
Impacted FilesCoverage ΔComplexity Δ
apps/dav/lib/Server.php47.79% <100%> (ø)22 <0> (ø)⬇️
apps/files_trashbin/lib/Trashbin.php72.46% <0%> (-0.25%)136% <0%> (ø)

@georgehrke

Copy link
Copy Markdown
MemberAuthor

@rullzer Is the subtree thing tested anywhere? Can't find something relevant in the ServerTest.php

@rullzer

Copy link
Copy Markdown
Member

@georgehrke not explicitly I think

@MorrisJobke
MorrisJobke merged commit 51d867c into masterJun 11, 2018
@MorrisJobke
MorrisJobke deleted the bugfix/noid/enable_caldav_for_public_calendars branch June 11, 2018 07:17
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@georgehrke@rullzer@MorrisJobke