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

Add rake task to migrate TopicalEventFeaturing #8438

Conversation

lauraghiorghisor-tw
Copy link
Contributor

@lauraghiorghisor-tw lauraghiorghisor-tw commented Nov 1, 2023

Migrating all TopicalEventFeaturing to use a new data class to manage their image assets. We are reusing FeaturedImageData as this intermediary class, in the same way AttachmentData works for attachments and their assets.

The rake task will create a new FeaturedImageData for each pre-existing image and populate its respective assets for each image variant.

Trello card

@lauraghiorghisor-tw lauraghiorghisor-tw force-pushed the prepare-topical-event-featuring-to-use-featured-image-data branch 2 times, most recently from 7ece486 to 2e9c990 Compare November 1, 2023 09:40
Migrating all TopicalEventFeaturing to use a new data class to manage their
image assets. We are reusing FeaturedImageData as this intermediary class,
in the same way AttachmentData works for attachments and their assets.

The rake task will create a new FeaturedImageData for each pre-existing image
and populate its respective assets for each image variant.
@lauraghiorghisor-tw lauraghiorghisor-tw force-pushed the prepare-topical-event-featuring-to-use-featured-image-data branch from 2e9c990 to 45c366f Compare November 2, 2023 16:59
@lauraghiorghisor-tw lauraghiorghisor-tw merged commit fc9294d into main Nov 2, 2023
15 checks passed
@lauraghiorghisor-tw lauraghiorghisor-tw deleted the prepare-topical-event-featuring-to-use-featured-image-data branch November 2, 2023 17:09
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