Skip to content

[stable32] #61609:moved/copied files open - #63453

Merged
blizzz merged 3 commits into
stable32from
fix/61609-copy-pageload
Oct 22, 2026
Merged

blizzz merged 3 commits into
stable32from
fix/61609-copy-pageload

Conversation

@samin-z

@samin-z samin-z commented Aug 21, 2026

Copy link
Copy Markdown

Summary

When files are copied/moved to a new folder, clicking on new folder opens instead of downloading.

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@samin-z
samin-z requested review from a team and skjnldsv as code owners August 21, 2026 11:47
@samin-z
samin-z requested review from kristian-zendato and sorbaugh and removed request for a team August 21, 2026 11:47
@github-actions github-actions Bot changed the title #61609:moved/copied files open [stable32] #61609:moved/copied files open Aug 21, 2026
@samin-z
samin-z requested a review from grnd-alt August 21, 2026 11:48

@solracsf solracsf 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.

🐙

@samin-z

samin-z commented Aug 28, 2026

Copy link
Copy Markdown
Author

/compile

@samin-z samin-z closed this Sep 4, 2026
@samin-z
samin-z force-pushed the fix/61609-copy-pageload branch from a880cf1 to 9e6f5db Compare September 4, 2026 08:28
@samin-z samin-z reopened this Sep 4, 2026
@samin-z
samin-z force-pushed the fix/61609-copy-pageload branch from 2cf36ec to 05ed7e9 Compare September 4, 2026 12:29

@provokateurin provokateurin 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.

You should target the master branch and backport from there, unless this issue really only affects stable32.

@samin-z

samin-z commented Sep 24, 2026

Copy link
Copy Markdown
Author

You should target the master branch and backport from there, unless this issue really only affects stable32.

@provokateurin , it is just a quick fix for stable32.

@provokateurin

Copy link
Copy Markdown
Member

So master to stable33 are fine?
Please put this kind of information into the PR description the next time, so it's clear right away.

@samin-z

samin-z commented Sep 24, 2026

Copy link
Copy Markdown
Author

So master to stable33 are fine? Please put this kind of information into the PR description the next time, so it's clear right away.

@provokateurin, yes all is good there.
Will do, thank you.

@samin-z
samin-z force-pushed the fix/61609-copy-pageload branch from 05ed7e9 to e7e8e46 Compare September 30, 2026 12:02
@samin-z
samin-z requested a review from solracsf September 30, 2026 14:51
@samin-z

samin-z commented Sep 30, 2026

Copy link
Copy Markdown
Author

@provokateurin hey
runner 7 is failing in login/login-redirect.cy.ts, it times out looking for the Enable profile checkbox and this is not related to the open folder change, can you please bypass that check and merge?

Signed-off-by: samin-z <samin.zavarkesh@gmail.com>
Signed-off-by: samin-z <samin.zavarkesh@gmail.com>
@samin-z
samin-z force-pushed the fix/61609-copy-pageload branch from 914ad8e to 4385429 Compare October 2, 2026 09:18
@provokateurin

Copy link
Copy Markdown
Member

@samin-z I don't have the permissions to do that. Please ask your Team Lead instead.

Signed-off-by: samin-z <samin.zavarkesh@gmail.com>
@samin-z
samin-z force-pushed the fix/61609-copy-pageload branch from 0f9eb60 to 5ff7222 Compare October 22, 2026 12:18
@blizzz
blizzz merged commit fcd083b into stable32 Oct 22, 2026
120 of 124 checks passed
@blizzz
blizzz deleted the fix/61609-copy-pageload branch October 22, 2026 13:43
@welcome

welcome Bot commented Oct 22, 2026

Copy link
Copy Markdown

Thanks for your first pull request and welcome to the community! Feel free to keep them coming! If you are looking for issues to tackle then have a look at this selection: https://github.com/nextcloud/server/issues?q=is%3Aopen+is%3Aissue+label%3A%22good+first+issue%22

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.

5 participants