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

Do not convert JPEG images into JPG #1904

Merged
merged 1 commit into from Jul 17, 2020

Conversation

tvdeyen
Copy link
Member

@tvdeyen tvdeyen commented Jul 17, 2020

What is this pull request for?

Images in JPEG format do not need to be converted into JPGs.

Checklist

  • I have followed Pull Request guidelines
  • I have added a detailed description into each commit message
  • I have added tests to cover this change

Images in JPEG format do not need to be converted into JPGs
@tvdeyen tvdeyen added this to the 5.0 milestone Jul 17, 2020
@tvdeyen tvdeyen requested a review from mamhoff July 17, 2020 11:25
@tvdeyen tvdeyen merged commit e034b8b into AlchemyCMS:master Jul 17, 2020
@tvdeyen tvdeyen deleted the do-not-convert-jpeg-into-jpg branch July 17, 2020 12:20
tvdeyen added a commit that referenced this pull request Jul 17, 2020
Images in JPEG format do not need to be converted into JPGs
bichinger-stm added a commit to bichinger/alchemy_cms that referenced this pull request Jul 17, 2020
…hemy_cms into active-storage-integration

* 'active-storage-integration' of github.com:bichinger/alchemy_cms:
  v5.0.0
  Do not convert JPEG images into JPG (AlchemyCMS#1904)
  Do not enable image cropper if file is missing (AlchemyCMS#1903)
  Always show original image as zoomed image (AlchemyCMS#1902)
  Rename Attachment#urlname into slug (AlchemyCMS#1848)
  Deprecate toolbar helper
  Bump version to 5.0.0.rc2
  Fix tag-list height
  Fix vertical position of site name in page tab
  Allow editing sites
  Fix missing help_text_text translations
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.

None yet

2 participants