Skip to content

Move version of images to ImageData from ImageUploader#11172

Draft
patrickpatrickpatrick wants to merge 2 commits intomainfrom
landing-pages-new-crop
Draft

Move version of images to ImageData from ImageUploader#11172
patrickpatrickpatrick wants to merge 2 commits intomainfrom
landing-pages-new-crop

Conversation

@patrickpatrickpatrick
Copy link
Contributor

@patrickpatrickpatrick patrickpatrickpatrick commented Feb 24, 2026

What

Move version creation to ImageData from ImageUploader

Why

Each image kind can now have the same name. Before versions were declared on the same ImageUploader class, which meant that if versions were specified with the same name in the config then they would conflict.

⚠️ This repo is Continuously Deployed: make sure you follow the guidance ⚠️

This application is owned by the Whitehall Experience team. Please let us know in #govuk-whitehall-experience-tech when you raise any PRs.

Follow these steps if you are doing a Rails upgrade.

@patrickpatrickpatrick patrickpatrickpatrick changed the base branch from main to alt-aspect-ratios February 24, 2026 14:01
@patrickpatrickpatrick patrickpatrickpatrick force-pushed the alt-aspect-ratios branch 2 times, most recently from 4dc8f26 to feb612e Compare February 24, 2026 14:07
@patrickpatrickpatrick patrickpatrickpatrick marked this pull request as draft February 24, 2026 14:07
@patrickpatrickpatrick patrickpatrickpatrick changed the title landing pages new crop Single image for Landing Page images Feb 24, 2026
@patrickpatrickpatrick patrickpatrickpatrick force-pushed the alt-aspect-ratios branch 6 times, most recently from c00ddf1 to 01b928b Compare February 24, 2026 15:25
@patrickpatrickpatrick patrickpatrickpatrick force-pushed the landing-pages-new-crop branch 2 times, most recently from 7aa6fb1 to 232d662 Compare February 24, 2026 15:28
@patrickpatrickpatrick patrickpatrickpatrick force-pushed the alt-aspect-ratios branch 11 times, most recently from f8b0e56 to 14cc63f Compare February 25, 2026 16:15
@patrickpatrickpatrick patrickpatrickpatrick force-pushed the alt-aspect-ratios branch 13 times, most recently from c824b18 to a1d81d0 Compare March 3, 2026 16:23
Base automatically changed from alt-aspect-ratios to main March 4, 2026 15:44
@patrickpatrickpatrick patrickpatrickpatrick force-pushed the landing-pages-new-crop branch 4 times, most recently from 9541b78 to ae6a2d9 Compare March 6, 2026 12:22
@patrickpatrickpatrick patrickpatrickpatrick changed the title Single image for Landing Page images Move version of images to ImageData from ImageUploader Mar 6, 2026
@patrickpatrickpatrick patrickpatrickpatrick force-pushed the landing-pages-new-crop branch 4 times, most recently from 0518f68 to a5656af Compare March 6, 2026 15:28
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