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

BugFix: missing jpg content type breaks corrupts presentation for Office365 #435

Merged

Conversation

antonandreyev
Copy link

By default we add only "jpeg" into presentation's content types (see makeXmlContTypes function) but Office365 definitely expects to see "jpg" also and consider some presentations as corrupted depending on certain pictures inserted.

@gitbrent gitbrent merged commit 1b1a967 into gitbrent:master Dec 12, 2018
@gitbrent
Copy link
Owner

Thanks @antonandreyev !

I have not experienced problems with images on office.com, so thanks for the information and the patch.

screen shot 2018-12-11 at 20 37 35

gitbrent added a commit that referenced this pull request Dec 12, 2018
ericrockey pushed a commit to WeTransfer/PptxGenJS that referenced this pull request Mar 24, 2020
…-type-missing

BugFix: missing jpg content type breaks corrupts presentation for Office365
ericrockey pushed a commit to WeTransfer/PptxGenJS that referenced this pull request Mar 24, 2020
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