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

Replace "on sale" flag by "price drop" #24109

Merged
merged 2 commits into from Apr 20, 2021
Merged

Conversation

marionf
Copy link
Contributor

@marionf marionf commented Apr 19, 2021

Questions Answers
Branch? develop
Description? Replace "on sale" flag by "price drop"
Type? improvement
Category? BO
BC breaks? no
Deprecations? no
Fixed ticket? Fixes #19145
How to test? Verify the wording in the BO product page > Princing tab, it should be "Display the "Price drop!" flag on the product page, and on product listings.". Check the box and verify the flag on the product page and product listing, it should be "Price drop!"
Possible impacts? nothing

This change is Reviewable

@marionf marionf requested a review from a team as a code owner April 19, 2021 10:08
@prestonBot prestonBot added develop Branch Improvement Type: Improvement labels Apr 19, 2021
@prestonBot
Copy link
Collaborator

This pull request seems to contain new translation strings. I have summarized them below to ease up review:

(Note: this is an automated message, but answering it will reach a real human)

@prestonBot prestonBot added the Waiting for wording Status: action required, waiting for wording label Apr 19, 2021
@marionf marionf changed the title Replace "on sale" by "price drop" Replace "on sale" flag by "price drop" Apr 19, 2021
@Progi1984
Copy link
Contributor

@Progi1984
Copy link
Contributor

@marionf Great job ;)

@Julievrz Julievrz added Wording ✔️ Status: check done, wording approved and removed Waiting for wording Status: action required, waiting for wording labels Apr 19, 2021
@Progi1984 Progi1984 added the Waiting for QA Status: action required, waiting for test feedback label Apr 20, 2021
@sarahdib
Copy link
Contributor

@marionf it's ok 👍🏻

Capture d’écran 2021-04-20 à 08 59 32

Capture d’écran 2021-04-20 à 08 59 54

Capture d’écran 2021-04-20 à 09 00 06

@sarahdib sarahdib added QA ✔️ Status: check done, code approved and removed Waiting for QA Status: action required, waiting for test feedback labels Apr 20, 2021
@prestonBot
Copy link
Collaborator

QA approved, well done! Message to the maintainers: do not forget to milestone it before the merge.

@Progi1984 Progi1984 merged commit ce8c733 into PrestaShop:develop Apr 20, 2021
@Progi1984
Copy link
Contributor

Thanks @marionf & @sarahdib

@Progi1984 Progi1984 added this to the 1.7.8.0 milestone Apr 20, 2021
@marionf marionf deleted the onsale branch April 20, 2021 07:41
@kpodemski
Copy link
Contributor

@PrestaShop/product-team @PrestaShop/prestashop-maintainers

IMO It's wrong way of fixing #19145. #19145 is about page title, not "On sale" as overall in the system.

It would be confusing for people after all these years to change this flag.

@marionf
Copy link
Contributor Author

marionf commented Apr 20, 2021

Hi @kpodemski

We have identified that the "on sale" flag is confusing, it can be translated in different ways according to the language.
Some people use it to put forward discounted products, while some use it to put forward end of stock products without discount.
This why we change it for "price drop" to put forward discounted products.

Also, we are going to use a new "Last chance" flag and page to put forward end of stock products without discount.

Anyway, these flags can always be translated by merchants through the BO if needed.

@Hlavtox
Copy link
Contributor

Hlavtox commented Apr 20, 2021

@kpodemski Exactly.

I want to see the first issue where people say: I checked "Prices drop" but the product is not displayed on prices-drop page, why is that?

Incorrect title on prices-drop controller was successfully fixed here #23858 and it should have ended by this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
develop Branch Improvement Type: Improvement QA ✔️ Status: check done, code approved Wording ✔️ Status: check done, wording approved
Projects
None yet
Development

Successfully merging this pull request may close these issues.

"Prices drop" page mislabeled title
8 participants