Skip to content

[Bug] Add video thumbnail status endpoint for Studio polling#1810

Draft
markus-moser wants to merge 1 commit into
2025.4from
3407-fix-video-thumbnail-status-endpoint
Draft

[Bug] Add video thumbnail status endpoint for Studio polling#1810
markus-moser wants to merge 1 commit into
2025.4from
3407-fix-video-thumbnail-status-endpoint

Conversation

@markus-moser
Copy link
Copy Markdown
Contributor

@markus-moser markus-moser commented May 5, 2026

Changes in this pull request

Resolves pimcore/studio-ui-bundle#3407

Adds GET /pimcore-studio/api/assets/{id}/video/thumbnail/{thumbnailName}/status returning {status: finished | inprogress | error | not_started}.

studio-ui-bundle polls this while a Video editable is in the in-progress state and reloads the editor when conversion finishes. No existing endpoint exposes this — BinaryService only uses the status internally to decide between streaming and throwing ElementProcessingNotCompletedException.

Additional info

Companion PRs:

🚧 Draft pending the companion PRs above.

@codacy-production
Copy link
Copy Markdown

codacy-production Bot commented May 5, 2026

Not up to standards ⛔

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

Studio needs a way to detect when an inprogress video thumbnail
finishes converting so it can refresh the editor automatically.
Today the only consumer of the thumbnail status is BinaryService,
which uses it internally to decide whether to stream or throw
ElementProcessingNotCompletedException - it isn't exposed to clients.

Adds GET /pimcore-studio/api/assets/{id}/video/thumbnail/{thumbnailName}/status
returning {status: finished | inprogress | error | not_started}.
studio-ui-bundle polls this endpoint while a Video editable is in
the in-progress state and reloads the editor when conversion finishes.

Resolves pimcore/studio-ui-bundle#3407
@markus-moser markus-moser force-pushed the 3407-fix-video-thumbnail-status-endpoint branch from 49f7109 to d1edb64 Compare May 5, 2026 14:36
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented May 5, 2026

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants