Skip to content

[_]: bugfix/Fix video thumbnail generation for video files#448

Open
CandelR wants to merge 1 commit intofeature/PB-6372-pull-to-refreshfrom
fix/fix-thumbnail-library
Open

[_]: bugfix/Fix video thumbnail generation for video files#448
CandelR wants to merge 1 commit intofeature/PB-6372-pull-to-refreshfrom
fix/fix-thumbnail-library

Conversation

@CandelR
Copy link
Copy Markdown
Contributor

@CandelR CandelR commented May 11, 2026

Video thumbnails were never generated during photo backup on iOS. react-native-create-thumbnailnative module received a file:// URI but then wrapped it again with [NSURL fileURLWithPath:], producing an invalid double-scheme URL (file:///file:///path/to/video.mov).
AVAsset silently failed to load it, the promise rejected, and the silent catch {} in uploadThumbnailForAsset swallowed the error, so no video ever got a thumbnail uploaded.

Fix

Added else if ([url_ hasPrefix:@"file://"]) branch to the vidURL construction inCreateThumbnail.m, mirroring the same pattern already used for image URLs.

@CandelR CandelR requested a review from TamaraFinogina May 11, 2026 08:22
@CandelR CandelR self-assigned this May 11, 2026
@CandelR CandelR added the enhancement New feature or request label May 11, 2026
@sonarqubecloud
Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants