-
Notifications
You must be signed in to change notification settings - Fork 271
Use thumbnail_url for thumbnail generation when present #675
Copy link
Copy link
Closed
WordPress/openverse-api
#1138Labels
✨ goal: improvementImprovement to an existing user-facing featureImprovement to an existing user-facing feature💻 aspect: codeConcerns the software code in the repositoryConcerns the software code in the repository🟨 priority: mediumNot blocking but should be addressed soonNot blocking but should be addressed soon🧱 stack: apiRelated to the Django APIRelated to the Django API
Metadata
Metadata
Labels
✨ goal: improvementImprovement to an existing user-facing featureImprovement to an existing user-facing feature💻 aspect: codeConcerns the software code in the repositoryConcerns the software code in the repository🟨 priority: mediumNot blocking but should be addressed soonNot blocking but should be addressed soon🧱 stack: apiRelated to the Django APIRelated to the Django API
Type
Fields
Give feedbackNo fields configured for issues without a type.
Projects
Status
✅ Done
Problem
Some providers (like SMK) may link to quite large images in their
image_url. This can cause unacceptably slow response times or even timeouts when generating thumbnails via our thumbnail service.Description
We already have a
thumbnail_urlavailable on the Image model which we can use.When the
thumbnail_urlis available, we should send this to the thumbnail service instead of theimage_url. This should be a small change here:(using the serialized field names)
Additional context
#1450 tracks updating the SMK provider script in the Catalog to populate
thumbnail_urlwith a link to a smaller image size.Implementation