Skip to content

fix: don't try to create preview folder if it already exists#58107

Merged
CarlSchwan merged 1 commit intomasterfrom
local-preview-mkdir-exists
Feb 13, 2026
Merged

fix: don't try to create preview folder if it already exists#58107
CarlSchwan merged 1 commit intomasterfrom
local-preview-mkdir-exists

Conversation

@icewind1991
Copy link
Member

The @ doesn't seem to be enough to reliably silence a warning being logged

Signed-off-by: Robin Appelman <robin@icewind.nl>
@icewind1991 icewind1991 added this to the Nextcloud 33 milestone Feb 5, 2026
@icewind1991 icewind1991 requested a review from a team as a code owner February 5, 2026 14:23
@icewind1991 icewind1991 added the 3. to review Waiting for reviews label Feb 5, 2026
@icewind1991 icewind1991 requested review from ArtificialOwl, CarlSchwan, leftybournes and salmart-dev and removed request for a team February 5, 2026 14:23
@AndyScherzinger
Copy link
Member

/backport to stable33

Copy link
Contributor

@szaimen szaimen left a comment

Choose a reason for hiding this comment

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

LGTM bit did not test

@CarlSchwan CarlSchwan merged commit 2391d58 into master Feb 13, 2026
203 of 215 checks passed
@CarlSchwan CarlSchwan deleted the local-preview-mkdir-exists branch February 13, 2026 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants