Matthieu Rolland

Results 51 comments of Matthieu Rolland
trafficstars

well it says you have to run `php install/upgrade/upgrade.php`, which should do the doctrine schema update... Maybe we should detail more clearly what it does ? Or add a section...

> @matthieu-rolland in that case, why #24948 failed? 🤔 Because he did the SQL updates manually, using the SQL update queries in: `install-dev/upgrade/sql/1.7.8.0.sql` He was missing columns added by doctrine...

> @matthieu-rolland There are different aspects: > > * `install-dev/upgrade/upgrade.php` does not work in 1.7.8.0-beta.1 > * It is important to know what it does, not only to be able...

> * It is important to know what it does, not only to be able to upgrade manually, but also to be aware, what (potentially destructive) SQL queries would be...

@khouloudbelguith it seems like the bug doesn't come from this PR, he fixed it here: https://github.com/PrestaShop/PrestaShop/pull/28747 (it's waiting for QA as of now) once it's merged this PR should work...

@PrestaShop/qa-team, @MatShir the issue said to wait for the fix on the core before using it in the module, but the fix is only available on prestashop 8.0 So for...

thank you @khouloudbelguith for the sum up !

About `upgrade de 1778 => 8.0.0: NOK` It will need to be tested with this PR, but with a release generated from [this PR on the core](https://github.com/PrestaShop/PrestaShop/pull/29469)

@khouloudbelguith now it should work when upgrading from 1778, I updated the `how to test` with a link to the core PR, which must used to generate the prestashop release

I fixed the conflict but can't push on the PR for some reason I get `permission denied`, did you change the rights on your repo @okom3pom ?