Skip to content

#61609: newly created folder gets downloaded #61610

Open
samin-z wants to merge 1 commit into
stable32from
fix/move-copy-files
Open

#61609: newly created folder gets downloaded #61610
samin-z wants to merge 1 commit into
stable32from
fix/move-copy-files

Conversation

@samin-z

@samin-z samin-z commented Jun 26, 2026

Copy link
Copy Markdown

Signed-off-by: samin-z <samin.zavarkesh@gmail.com>
@samin-z samin-z requested review from a team and skjnldsv as code owners June 26, 2026 11:26
@samin-z samin-z requested review from nfebe and sorbaugh and removed request for a team June 26, 2026 11:26

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This looks unrelated and rather a UI feature that must go to master first

@susnux susnux left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This does not match the library version for Nextcloud 32.
Only in Nextcloud 34 the action format was changed (as we updated to v4 of the library)

@@ -243,7 +277,7 @@ export default defineComponent({
// In case something goes wrong, since we don't want to break
// the entire list, we filter out actions that throw an error.
try {
return action.enabled([this.source], this.currentView)
return (action.enabled as (context: FileActionContext) => boolean)(this.fileActionContext)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

thank you for the comments @susnux
i'll update accordingly.

@solracsf solracsf linked an issue Jun 27, 2026 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Stable 32: newly created folder gets downloaded

2 participants