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

Disable modules with 1.6 method #86

Merged
merged 2 commits into from Jun 22, 2018

Conversation

Projects
None yet
1 participant
@Quetzacoalt91
Member

Quetzacoalt91 commented Jun 22, 2018

It appeared disabling modules triggered many errors during the upgrade file, as we were trying to find which one had to be disabled. To find these, we had to instantiate them, generally throwing errors on the new core.

From this PR, we try to use the 1.6 method, which only reads the config.xml files.

@Quetzacoalt91 Quetzacoalt91 changed the title from Disable modules before file upgrade to Disable modules with 1.6 method Jun 22, 2018

@Quetzacoalt91 Quetzacoalt91 added this to the 4.0.0 milestone Jun 22, 2018

@Quetzacoalt91 Quetzacoalt91 merged commit 0b084bd into PrestaShop:dev Jun 22, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@Quetzacoalt91 Quetzacoalt91 deleted the Quetzacoalt91:disable-modules-before-file-upgrade branch Jun 22, 2018

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