Skip to content

Do not allow transfer ownership when the user isn't the owner - #19764

Merged
rullzer merged 1 commit into
masterfrom
fix/transfer-ownerhip-owner-check
Mar 3, 2020
Merged

Do not allow transfer ownership when the user isn't the owner#19764
rullzer merged 1 commit into
masterfrom
fix/transfer-ownerhip-owner-check

Conversation

@ChristophWurst

Copy link
Copy Markdown
Member

No description provided.

Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>

@rullzerrullzer left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice and clean

@gary-kimgary-kim left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@gary-kimgary-kim added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Mar 3, 2020
@rullzer
rullzer merged commit 65b75c8 into masterMar 3, 2020
@rullzer
rullzer deleted the fix/transfer-ownerhip-owner-check branch March 3, 2020 19:20
@rullzer

Copy link
Copy Markdown
Member

/backport to stable18

@backportbot-nextcloud

Copy link
Copy Markdown

The backport to stable18 failed. Please do this backport manually.

@ChristophWurst

Copy link
Copy Markdown
MemberAuthor

Backport at #19773

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

Labels

4. to releaseReady to be released and/or waiting for tests to finishfeature: filesfeature: users and groups

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@ChristophWurst@rullzer@gary-kim