-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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
feat(trashbin): Add deleted by properties #44643
Conversation
c66be6e
to
3b338fe
Compare
This comment was marked as resolved.
This comment was marked as resolved.
3b338fe
to
ab46e8a
Compare
ab46e8a
to
7b62df5
Compare
d1672d9
to
1c3b229
Compare
3b46ab2
to
6e49424
Compare
Switched the column back to nullable to fix potential migration errors with oracle which converts empty string to null |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you test that with groupfolders?
Signed-off-by: Christopher Ng <chrng8@gmail.com>
Signed-off-by: Christopher Ng <chrng8@gmail.com>
Signed-off-by: Christopher Ng <chrng8@gmail.com>
Signed-off-by: Christopher Ng <chrng8@gmail.com>
6e49424
to
583517e
Compare
Seems it breaks groupfolders as it implements a custom trash backend, documented the change here nextcloud/documentation#11757 App devs will have to update their custom trash backends to support this for 30 |
Summary
Store and provide a new
trashbin-deleted-by-id
andtrashbin-deleted-by-display-name
DAV propertiesExample PROPFIND request data
Checklist