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

Promotions aren't advertised? #3534

Closed
cdrage opened this issue Feb 29, 2020 · 3 comments
Closed

Promotions aren't advertised? #3534

cdrage opened this issue Feb 29, 2020 · 3 comments

Comments

@cdrage
Copy link

cdrage commented Feb 29, 2020

Solidus Version:

Master branch

To Reproduce

To reproduce:

  1. Add a promotion
  2. Create a rule to select an item
  3. Visit the homepage (there is no promotions being shown)

Current behavior

When adding a promotion, there's no way to 'advertise' it on the main page or even the product..

I see that there are advertise code all throughout solidus, but there is no button to activate it?

Expected behavior

Shows the advertisement on the product / homepage.

Screenshots

N/A. It's just not there.

Desktop (please complete the following information):

  • OS: [e.g. iOS]
  • Browser [e.g. chrome, safari]
  • Version [e.g. 22]

Smartphone (please complete the following information):

  • Device: [e.g. iPhone6]
  • OS: [e.g. iOS8.1]
  • Browser [e.g. stock browser, safari]
  • Version [e.g. 22]

Additional context

@peterberkenbosch
Copy link
Contributor

Hi Charlie, this has been removed from master some time ago. also see #2737

@seandawes
Copy link

@cdrage We implemented some front end simple logic that may be of interest here and to community as its pretty common on any platform. If an item is marked on sale on product admin, on front end the product is tagged with a SALE banner in product listing / category pages in addition to the product page itself

Example category page

image

We much rather create front end discounts than insist users use coupon codes for the manufacturer based sales generally. Unless in fact you need to add an additional layer of tracking in certain marketing instances that warrant it. (ie. using unique discount codes)

@jarednorman
Copy link
Member

jarednorman commented Apr 24, 2020

This functionality is technically used in the default frontend. The product show page renders this partial which relies on Spree::Product#possible_promotions.

The partial shows a list of links to products that are part of promotions this product is part of. I tend to think we should just remove the advertise functionality as the promotions partial is confusing from a customer perspective (it's just a list of products with the title "Promotions" with no explanation) and the functionality to create "advertised" promotions was removed from the admin ages ago.

Stores generally have broader promotion advertising needs than we would likely want to build into the frontend, so I'm for deprecating and removing the remaining functionality that relies on this feature. Documenting strategies for adding visibility to your promotions is something I think a lot of stores would benefit from though.

@cdrage cdrage closed this as completed Mar 29, 2022
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

No branches or pull requests

4 participants