Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Am/backport 2024 commits #1

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from
Open

Conversation

sta1r
Copy link

@sta1r sta1r commented Jan 16, 2025

This PR backports some 2024 changes we originally made to a copy of the plugin, stored in private version control.

sta1r added 4 commits January 16, 2025 16:36
…acking generated thumbnails

## What's being changed
The windows azure storage plugin file.

## Why it's being changed
The code has been changed so that it doesn't throw an Exception for missing thumbnails if the mime type matches 'application/pdf'. Editors uploading PDFs into the media library were seeing an error, and the assets were not transferred to the CDN.
## How to review / test this change
- Go to the Media Library
- Upload a PDF
- See no errors
- Confirm the PDF can be downloaded from the CDN

Related work items: #273868
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.

1 participant