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 a float parser for the wholesale price of attributes #26188

Merged

Conversation

andromaque
Copy link
Contributor

@andromaque andromaque commented Oct 11, 2021

Questions Answers
Branch? develop
Description? Add a float parser for the wholesale price of attributes
Type? bug fix
Category? BO
BC breaks? no
Deprecations? no
Fixed ticket? Fixes #23922.
How to test? From the Back-Office Catalog / Products, click to edit a Product with combinations
Go to the Combinations tab, and then edit a combination
Enter a wholesale price for the combination (e.g. 1.55)
Save and go back to the product listing
Click to edit the same product
Click on Save and go back to the product listing
Click to edit the same product (for the 3rd time), go to the Combinations tab and click on the combination: the wholesale price is not rounded anymore.
Possible impacts? N/A

This change is Reviewable

Add a float parser for the wholesale price of attributes
@andromaque andromaque requested a review from a team as a code owner October 11, 2021 07:22
@prestonBot prestonBot added develop Branch Bug fix Type: Bug fix labels Oct 11, 2021
@Progi1984 Progi1984 changed the title Andromaque patch wholesale price Add a float parser for the wholesale price of attributes Oct 11, 2021
@PierreRambaud PierreRambaud added the Waiting for QA Status: action required, waiting for test feedback label Oct 13, 2021
@marwachelly marwachelly self-assigned this Oct 27, 2021
@marwachelly
Copy link

Hello,
PR is tested and the problem is fixed. The wholesale price stay the same when the product is edited.
https://www.awesomescreenshot.com/video/5869060?key=1fd73801ef8dc70661b2ef53f1c93445
Thanks.

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

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

@Progi1984 Progi1984 closed this Nov 2, 2021
@Progi1984 Progi1984 reopened this Nov 2, 2021
@Progi1984 Progi1984 added this to the 8.0.0 milestone Nov 3, 2021
@Progi1984 Progi1984 merged commit fe8bc55 into PrestaShop:develop Nov 3, 2021
@prestonBot
Copy link
Collaborator

Contribution merged, congratulations!

Would you mind answering our quick 1-minute survey? We would love to hear about your experience so far, it will help us improve our process for the community involved, like you. ;-)

@Progi1984
Copy link
Contributor

Thanks @andromaque & @marwachelly

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

Successfully merging this pull request may close these issues.

The decimal of wholesale prices on attributes is lost when editing the product
7 participants