Skip to content

Conversation

@DeepDiver1975
Copy link
Member

No description provided.

@DeepDiver1975 DeepDiver1975 added this to the 9.2 milestone Jul 26, 2016
@mention-bot
Copy link

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

@DeepDiver1975
Copy link
Member Author

@SergioBertolinSG we might want to think about an integration test for this one - THX

When Sending a "PROPFIND" to "/remote.php/dav/files/admin/welcome.txt" with requesttoken
Then the HTTP status code should be "207"

Scenario: Uploading a file as recipient using webdav new endpoint having quota
Copy link
Member Author

Choose a reason for hiding this comment

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

while this scenario is still valid - it does not match this pr.

The scenario should look like:

  • user0 has 0B quota
  • upload of file with 10B fails with 207

Copy link
Contributor

Choose a reason for hiding this comment

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

507 is the expected code isn't?

@DeepDiver1975
Copy link
Member Author

thx @SergioBertolinSG

@DeepDiver1975
Copy link
Member Author

👍

@DeepDiver1975 DeepDiver1975 merged commit 4374b46 into master Jul 29, 2016
@DeepDiver1975 DeepDiver1975 deleted the add-quotaplugin-v2-master branch July 29, 2016 04:40
DeepDiver1975 added a commit that referenced this pull request Jul 29, 2016
* Adding quota plugin to new dav endpoint

* Added integrated test failing in old endpoint

* Added 0B quota test
DeepDiver1975 added a commit that referenced this pull request Jul 29, 2016
* Adding quota plugin to new dav endpoint

* Added integrated test failing in old endpoint

* Added 0B quota test
DeepDiver1975 added a commit that referenced this pull request Jul 31, 2016
* Adding quota plugin to new dav endpoint

* Added integrated test failing in old endpoint

* Added 0B quota test
DeepDiver1975 added a commit that referenced this pull request Aug 1, 2016
* Adding quota plugin to new dav endpoint

* Added integrated test failing in old endpoint

* Added 0B quota test
@lock
Copy link

lock bot commented Aug 5, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Aug 5, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants