Skip to content

[stable33] fix(preview): First cleanup from filecache and then from preview table - #63006

Open
backportbot[bot] wants to merge 6 commits into
stable33from
backport/61531/stable33
Open

[stable33] fix(preview): First cleanup from filecache and then from preview table#63006
backportbot[bot] wants to merge 6 commits into
stable33from
backport/61531/stable33

Conversation

@backportbot

@backportbotbackportbotBot commented Aug 7, 2026

Copy link
Copy Markdown

Backport of #61531

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

Todo

  • Review and resolve any conflicts
  • Review and verify the backported changes
  • Remove all the empty commits
  • Amend HEAD commit to remove the line stating to skip CI

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

Signed-off-by: Carl Schwan <carlschwan@kde.org>
Signed-off-by: Carl Schwan <carlschwan@kde.org>
Signed-off-by: Carl Schwan <carlschwan@kde.org>
Now the logic is inverted so the tests need to be changed
Signed-off-by: Carl Schwan <carlschwan@kde.org>
Signed-off-by: Carl Schwan <carlschwan@kde.org>
If we don't find previews in the filecache, this is now normal. Don't
abort and instead delete previews from the new preview table instead.
Signed-off-by: Carl Schwan <carlschwan@kde.org>
@CarlSchwan
CarlSchwanforce-pushed the backport/61531/stable33 branch from bef0841 to 35257f0CompareAugust 10, 2026 09:23
@CarlSchwan
CarlSchwan marked this pull request as ready for review August 10, 2026 09:23
@CarlSchwan
CarlSchwan requested a review from a team as a code ownerAugust 10, 2026 09:23
@CarlSchwan
CarlSchwan requested review from come-nc, icewind1991, leftybournes and salmart-dev and removed request for a teamAugust 10, 2026 09:23
@CarlSchwanCarlSchwan changed the title [stable33] [stable34] fix(preview): First cleanup from filecache and then from preview table[stable33] fix(preview): First cleanup from filecache and then from preview tableAug 10, 2026
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@artonge@CarlSchwan