Skip to content

Conversation

@LukasReschke
Copy link
Member

Downstreaming of owncloud/core#25683

Signed-off-by: Lukas Reschke lukas@statuscode.ch

@LukasReschke LukasReschke added 3. to review Waiting for reviews downstream labels Sep 26, 2016
@LukasReschke LukasReschke added this to the Nextcloud 11.0 milestone Sep 26, 2016
@mention-bot
Copy link

@LukasReschke, thanks for your PR! By analyzing the annotation information on this pull request, we identified @blizzz, @nickvergessen and @MorrisJobke to be potential reviewers

@nickvergessen
Copy link
Member

👍

@MorrisJobke
Copy link
Member

Tested and works 👍 Header is now included

Downstreaming of owncloud/core#25683

Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
@LukasReschke
Copy link
Member Author

Rebased upon master.

@codecov-io
Copy link

Current coverage is 30.68% (diff: 64.28%)

Merging #1513 into master will increase coverage by <.01%

@@             master      #1513   diff @@
==========================================
  Files          1080       1080          
  Lines         59827      59836     +9   
  Methods        6771       6772     +1   
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits          18353      18361     +8   
- Misses        41474      41475     +1   
  Partials          0          0          

Sunburst

Diff Coverage File Path
0% apps/dav/lib/Server.php
•••••••••• 100% ...pps/dav/lib/Connector/Sabre/CopyEtagHeaderPlugin.php

Powered by Codecov. Last update 2eaa279...bd96c6a

@MorrisJobke
Copy link
Member

Retested and works 👍

@MorrisJobke MorrisJobke merged commit 85301df into master Sep 28, 2016
@MorrisJobke MorrisJobke deleted the fix-etag-on-move branch September 28, 2016 19:46
@MorrisJobke
Copy link
Member

This was backported upstream - owncloud/core#26334

Should we also backport this?

@LukasReschke
Copy link
Member Author

I guess so, assuming the clients use that probably 🙈

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

Labels

3. to review Waiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants