Skip to content

Extend ViewOnly DAV plugin to versions endpoint - #36113

Merged
PVince81 merged 1 commit into
masterfrom
bugfix/noid/viewonly-versions
Jan 16, 2023
Merged

Extend ViewOnly DAV plugin to versions endpoint#36113
PVince81 merged 1 commit into
masterfrom
bugfix/noid/viewonly-versions

Conversation

@PVince81

@PVince81 PVince81 commented Jan 12, 2023

Copy link
Copy Markdown
Member

Summary

Applies the ViewOnly DAV plugin also to the versions DAV endpoint

Checklist

@PVince81 PVince81 added the 3. to review Waiting for reviews label Jan 12, 2023
@PVince81 PVince81 added this to the Nextcloud 26 milestone Jan 12, 2023
@PVince81 PVince81 self-assigned this Jan 12, 2023
@PVince81

PVince81 commented Jan 12, 2023

Copy link
Copy Markdown
Member Author

/backport to stable25

github-advanced-security[bot]
github-advanced-security AI found potential problems Jan 12, 2023
View reviewed changes
Comment thread apps/dav/lib/DAV/ViewOnlyPlugin.php Fixed
@PVince81
PVince81 force-pushed the bugfix/noid/viewonly-versions branch from d6eb439 to 78f53cc Compare January 12, 2023 11:09
@juliusknorr juliusknorr added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Jan 12, 2023
artonge
artonge approved these changes Jan 12, 2023
View reviewed changes
Comment thread apps/dav/lib/DAV/ViewOnlyPlugin.php Outdated
@PVince81
PVince81 force-pushed the bugfix/noid/viewonly-versions branch from 78f53cc to 71a63d9 Compare January 12, 2023 12:45
@PVince81

PVince81 commented Jan 13, 2023

Copy link
Copy Markdown
Member Author

/rebase

@PVince81
Extend ViewOnly DAV plugin to versions endpoint
ecf6b76 
Signed-off-by: Vincent Petry <vincent@nextcloud.com>
@nextcloud-command
nextcloud-command force-pushed the bugfix/noid/viewonly-versions branch from 71a63d9 to ecf6b76 Compare January 13, 2023 09:27
@PVince81
PVince81 merged commit 60eac3f into master Jan 16, 2023
@PVince81
PVince81 deleted the bugfix/noid/viewonly-versions branch January 16, 2023 13:09
@backportbot-nextcloud backportbot-nextcloud Bot mentioned this pull request Jan 16, 2023
Merged
@PVince81

PVince81 commented Jan 23, 2023

Copy link
Copy Markdown
Member Author

/backport to stable24

@backportbot-nextcloud backportbot-nextcloud Bot mentioned this pull request Jan 23, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nickvergessen nickvergessen nickvergessen approved these changes
@juliusknorr juliusknorr juliusknorr approved these changes
@artonge artonge artonge approved these changes
@blizzz blizzz Awaiting requested review from blizzz
@come-nc come-nc Awaiting requested review from come-nc

Assignees

@PVince81 PVince81

Labels

4. to release Ready to be released and/or waiting for tests to finish

Projects

None yet

Milestone

Nextcloud 26

Development

Successfully merging this pull request may close these issues.

6 participants

@PVince81 @nickvergessen @juliusknorr @artonge @github-advanced-security @skjnldsv