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

fix(storefront): BCTHEME-1481 fix sales badges #2352

Closed

Conversation

bc-alexsaiannyi
Copy link
Contributor

@bc-alexsaiannyi bc-alexsaiannyi commented Apr 26, 2023

What?

This PR fixes sale badges appearance and and modifies logic for sold-out badges as well that appears after changes in BCTHEME-1171.

Requirements

  • CHANGELOG.md entry added (required for code changes only)

Tickets / Documentation

Screenshots (if appropriate)

TBD

@bc-alexsaiannyi
Copy link
Contributor Author

Before reviewing/merging PR we need to change a show_cart_action for more appropriate flag in badges logic. FYI, it's been used for checking if a product is purchasable on PLP.

Screenshot with this setting in CP is below.
Screenshot 2023-04-26 at 12 26 54

@bc-azvierieva
Copy link

Closed as this issue will be fixed as part of revert for fix BCTHEME-1171 #2354

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