Skip to content

Conversation

@CarlSchwan
Copy link
Member

@CarlSchwan CarlSchwan commented Oct 9, 2025

Summary

When requesting previews, which we don't find in oc_previews, search in IAppData first before creating them.

Move the logic from MovepreviewJob to PreviewMigrationService and reuse that in the Preview Generator.

At the same time rename MovePreviewJob to PreviewMigrationJob as it is a better name.

Checklist

@CarlSchwan CarlSchwan force-pushed the carl/on-demand-preview-migration branch 5 times, most recently from 675b624 to a0d81ae Compare October 9, 2025 15:39
@CarlSchwan CarlSchwan force-pushed the carl/on-demand-preview-migration branch from a0d81ae to abc8733 Compare January 12, 2026 12:50
@CarlSchwan CarlSchwan marked this pull request as ready for review January 12, 2026 12:55
@CarlSchwan CarlSchwan requested a review from a team as a code owner January 12, 2026 12:55
@CarlSchwan CarlSchwan requested review from icewind1991, nfebe, sorbaugh and yemkareems and removed request for a team January 12, 2026 12:55
When requesting previews, which we don't find in oc_previews, search in
IAppData first before creating them.

Move the logic from MovepreviewJob to PreviewMigrationService and reuse
that in the Preview Generator.

At the same time rename MovePreviewJob to PreviewMigrationJob as it is a
better name.

Signed-off-by: Carl Schwan <carl.schwan@nextcloud.com>
@CarlSchwan CarlSchwan force-pushed the carl/on-demand-preview-migration branch from 66a2427 to 4d4ead5 Compare January 12, 2026 13:20
Signed-off-by: Carl Schwan <carlschwan@kde.org>
@CarlSchwan CarlSchwan force-pushed the carl/on-demand-preview-migration branch from 4d4ead5 to 7a025ff Compare January 14, 2026 13:09
@CarlSchwan CarlSchwan added this to the Nextcloud 33 milestone Jan 14, 2026
@CarlSchwan CarlSchwan self-assigned this Jan 14, 2026
@CarlSchwan CarlSchwan added the 3. to review Waiting for reviews label Jan 14, 2026
@nextcloud-bot nextcloud-bot mentioned this pull request Jan 14, 2026
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