Skip to content
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

[stable29] fix(files): hide move dialog action if destination has no CREATE permission #50238

Open
wants to merge 4 commits into
base: stable29
Choose a base branch
from

Conversation

backportbot[bot]
Copy link

@backportbot backportbot bot commented Jan 17, 2025

Backport of #45982
Backport of #50177
Backport of #50206

Warning, This backport's changes differ from the original and might be incomplete ⚠️

@backportbot backportbot bot requested a review from skjnldsv as a code owner January 17, 2025 15:03
@backportbot backportbot bot requested review from susnux and Antreesy January 17, 2025 15:03
@backportbot backportbot bot added this to the Nextcloud 29.0.12 milestone Jan 17, 2025
@Antreesy Antreesy force-pushed the backport/50177/stable29 branch from 764a649 to 158872d Compare January 17, 2025 17:05
@Antreesy

This comment was marked as outdated.

@Antreesy Antreesy force-pushed the backport/50177/stable29 branch from 6e78683 to cd31ec2 Compare January 21, 2025 11:46
@Antreesy
Copy link
Contributor

/compile /

@Antreesy Antreesy force-pushed the backport/50177/stable29 branch from 4b56a1c to 7653f96 Compare January 21, 2025 12:41
@Antreesy
Copy link
Contributor

CI Fail is due to #47185 backport is missing.
If there's no plan to backport it to stable29, shall we close this? @susnux @skjnldsv

Only missing bit is this one, though I don't know how much else will be broken:

export const canMove = (nodes: Node[]) => {
	const minPermission = nodes.reduce((min, node) => Math.min(min, node.permissions), Permission.ALL)
	return Boolean(minPermission & Permission.DELETE) // Previously Permission.UPDATE
}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants