-
Notifications
You must be signed in to change notification settings - Fork 156
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
Send mtime with uploads #3377
Send mtime with uploads #3377
Conversation
Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes. |
Note: with TUS upload (chunked upload) we already send the mtime. |
bb6817f
to
57f92db
Compare
💥 Acceptance tests webUIOCIS failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/phoenix/9531/
|
💥 Acceptance tests webUIOCIS failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/phoenix/9548/
|
Strange. Uploading works for me with manual upload also with OCIS. Next up to try:
|
57f92db
to
944971e
Compare
fixed the minor comment and rebased. |
💥 Acceptance tests webUIOCIS failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/phoenix/9653/
|
Aligns with the behavior of OC 10 which itself was aligned with the behavior of the desktop client.
944971e
to
2cb8266
Compare
finally tests passed. I guess the side effects might have been resolved with recent work on stabilizing upload tests. |
Description
Aligns with the behavior of OC 10 which itself was aligned with the
behavior of the desktop client.
Related Issue
Fixes #2969
Motivation and Context
How Has This Been Tested?
See #2969
Screenshots (if appropriate):
Types of changes
Checklist:
Open tasks: