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

Removed duplicate line in upgrade file #8643

Merged
merged 1 commit into from Dec 29, 2017

Conversation

Projects
None yet
2 participants
@eternoendless
Member

eternoendless commented Dec 28, 2017

Questions Answers
Branch? 1.7.3.x
Description? Removed a duplicate line in 1.7.3.0.sql
Type? bug fix
Category? CO
BC breaks? no
Deprecations? no
Fixed ticket? n/a
How to test? No testing needed

This change is Reviewable

@eternoendless eternoendless added this to the 1.7.3.0 milestone Dec 28, 2017

@@ -24,7 +24,6 @@ ALTER TABLE `PREFIX_product_shop` ADD `low_stock_alert` TINYINT(1) NOT NULL DEFA
ALTER TABLE `PREFIX_product_attribute` ADD `low_stock_alert` TINYINT(1) NOT NULL DEFAULT 0 AFTER `low_stock_threshold`;
ALTER TABLE `PREFIX_product_attribute_shop` ADD `low_stock_alert` TINYINT(1) NOT NULL DEFAULT 0 AFTER `low_stock_threshold`;
ALTER TABLE `PREFIX_product_attribute_shop` ADD `low_stock_threshold` INT(10) NULL DEFAULT NULL AFTER `minimal_quantity`;

This comment has been minimized.

@eternoendless

eternoendless Dec 28, 2017

Member

Already done in line 20

@eternoendless

eternoendless Dec 28, 2017

Member

Already done in line 20

@Quetzacoalt91 Quetzacoalt91 merged commit 84b4808 into PrestaShop:1.7.3.x Dec 29, 2017

2 checks passed

Codacy/PR Quality Review Good work! A positive pull request.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@Quetzacoalt91

This comment has been minimized.

Show comment
Hide comment
@Quetzacoalt91
Member

Quetzacoalt91 commented Dec 29, 2017

Thank you @eternoendless

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