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

@backportbot backportbot Bot 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>
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
CarlSchwan force-pushed the backport/61531/stable33 branch from bef0841 to 35257f0 Compare August 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 owner August 10, 2026 09:23
@CarlSchwan
CarlSchwan requested review from come-nc, icewind1991, leftybournes and salmart-dev and removed request for a team August 10, 2026 09:23
@CarlSchwan
CarlSchwan enabled auto-merge August 10, 2026 09:23
@CarlSchwan CarlSchwan 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 table Aug 10, 2026
Sign up for free to 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