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

Use configurable adjustment promotion source types in Thumbnail.for #5781

Merged
merged 1 commit into from
Jun 11, 2024

Conversation

mamhoff
Copy link
Contributor

@mamhoff mamhoff commented Jun 7, 2024

There's a View Component for rendering icons, and it's got a convenience method for rendering adjustables to adjustments.

This uses the Spree::Config.adjustment_promotion_source_types configuration option to figure out whether an adjustable needs a megaphone as an icon.

With the legacy promotion system extracted, Spree::PromotionAction will not necessarily be available.

@github-actions github-actions bot added changelog:solidus_core Changes to the solidus_core gem changelog:solidus_admin labels Jun 7, 2024
There's a View Component for rendering icons, and it's got a convenience
method for rendering adjustables to adjustments.

This uses the `Spree::Config.adjustment_promotion_source_types`
configuration option to figure out whether an adjustable needs a
megaphone as an icon.

With the legacy promotion system extracted, `Spree::PromotionAction` will not
necessarily be available.
@mamhoff mamhoff force-pushed the configurable-promotion-thumbnails branch from 5bc370a to f326ca4 Compare June 7, 2024 12:30
@github-actions github-actions bot removed the changelog:solidus_core Changes to the solidus_core gem label Jun 7, 2024
@mamhoff mamhoff marked this pull request as ready for review June 7, 2024 12:34
@mamhoff mamhoff requested a review from a team as a code owner June 7, 2024 12:34
@tvdeyen tvdeyen merged commit ca1a3b5 into solidusio:main Jun 11, 2024
11 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants