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 delivery time not shown #9219

Merged
merged 1 commit into from Jun 27, 2018

Conversation

Projects
None yet
4 participants
@JuanjoSCu
Contributor

JuanjoSCu commented Jun 26, 2018

To solve http://forge.prestashop.com/browse/BOOM-5829

Questions Answers
Branch? develop
Description? If we have a product backordered, stock -1 or less the text with the transport delivery time does not show.
Type? bug fix
Category? FO
BC breaks? no
Deprecations? no
Fixed ticket? http://forge.prestashop.com/browse/BOOM-5829
How to test? see that a product with stock -1 does not appear the delivery time

Important guidelines


This change is Reviewable

@prestonBot

This comment has been minimized.

Collaborator

prestonBot commented Jun 26, 2018

Hello @JuanjoSCu!

This is your first pull request on the PrestaShop project. Thank you, and welcome to this Open Source community!

@mickaelandrieu

This comment has been minimized.

Contributor

mickaelandrieu commented Jun 27, 2018

Hello @eternoendless, we just talked about this bug today, should we make the patch available in 1.7.4.x?

Welcome and thanks for this first contribution @JuanjoSCu!

@mickaelandrieu

This comment has been minimized.

Contributor

mickaelandrieu commented Jun 27, 2018

Thanks @JuanjoSCu !

@mickaelandrieu mickaelandrieu merged commit 83cac86 into PrestaShop:develop Jun 27, 2018

2 checks passed

Codacy/PR Quality Review Up to standards. A positive pull request.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@eternoendless eternoendless changed the title from Update product-prices.tpl to Fix delivery time not shown Aug 30, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment