Skip to content

Add missing changes for PS 9.0.1 and 9.0.0 (with catch-up script)#1484

Merged
Quetzacoalt91 merged 2 commits intoPrestaShop:devfrom
Quetzacoalt91:add-missing-changes-for-ps-9.0.1
Oct 7, 2025
Merged

Add missing changes for PS 9.0.1 and 9.0.0 (with catch-up script)#1484
Quetzacoalt91 merged 2 commits intoPrestaShop:devfrom
Quetzacoalt91:add-missing-changes-for-ps-9.0.1

Conversation

@Quetzacoalt91
Copy link
Member

@Quetzacoalt91 Quetzacoalt91 commented Oct 2, 2025

Questions Answers
Description? Update scripts for PS 9.0.0 and 9.0.1 with the differences found when comparing a updated shop and a fresh installation.
Type? improvement
BC breaks? Nope
Deprecations? Nope
Fixed ticket? /
Sponsor company @PrestaShopCorp
How to test? When using https://github.com/PrestaShop/SeamlessUpgradeToolbox/tree/main/autoupgrade-scripts, we should not find differences in the hooks of the database structure.

Related PRs:

PrestaShop 9.0.0

PrestaShop 9.0.1

@Quetzacoalt91 Quetzacoalt91 self-assigned this Oct 2, 2025
@Quetzacoalt91 Quetzacoalt91 added the enhancement Type: Improvement label Oct 2, 2025
@github-project-automation github-project-automation bot moved this to Ready for review in PR Dashboard Oct 2, 2025
@Quetzacoalt91 Quetzacoalt91 added this to the 7.4.1 milestone Oct 2, 2025
@Quetzacoalt91 Quetzacoalt91 force-pushed the add-missing-changes-for-ps-9.0.1 branch from 6e2bc06 to fda725a Compare October 2, 2025 16:51
ga-devfront
ga-devfront previously approved these changes Oct 6, 2025
Copy link
Contributor

@ga-devfront ga-devfront left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR it's good for me, I just added several comments.

@ingridusta ingridusta self-assigned this Oct 7, 2025
@Quetzacoalt91 Quetzacoalt91 force-pushed the add-missing-changes-for-ps-9.0.1 branch from 660f93e to d22998f Compare October 7, 2025 13:19
@sonarqubecloud
Copy link

sonarqubecloud bot commented Oct 7, 2025

Copy link

@ingridusta ingridusta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @Quetzacoalt91,

From ur branch, when i update to 9.0.1, I don't have the hooks "actionSubmitAccountBefore" and "actionGetPdfTemplateObject" ❌

Capture d’écran 2025-10-07 à 15 42 09

The index "id_product" is created.

@ingridusta ingridusta dismissed their stale review October 7, 2025 13:51

Sorry i didn't check the BDD entirely. I keep running tests.

Copy link

@ingridusta ingridusta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @Quetzacoalt91,

I tested the 3 cases :

🔹 This branch from 8.2.3 to 9.0.0 :
branch from 8 2 3 to 9 0 0 index
branch from 8 2 3 to 9 0 0 hook 1

🔹 This branch from 8.2.3 to 9.0.1 :
branch from 8 2 3 to 9 0 1 hook 1
branch from 8 2 3 to 9 0 1 hook 2
Capture d’écran 2025-10-07 à 16 20 26

🔹 Dev branch from 8.2.3 to 9.0.1 (no impacts) then, this branch to 9.0.1 :
from 8.2.3 to 9.0.1 :
dev branch after update to 9 0 0 _ hook
dev branch after update to 9 0 0 _ index no
this branch to 9.0.1 :
branch after update to v9 0 1 hook 1
branch after update to v9 0 1 hook 2

It's QA approve ✅

@ingridusta ingridusta added QA ✔️ Status: Check done, Code approved and removed waiting for QA labels Oct 7, 2025
@Quetzacoalt91 Quetzacoalt91 merged commit cffd71f into PrestaShop:dev Oct 7, 2025
52 of 54 checks passed
@github-project-automation github-project-automation bot moved this from Ready for review to Merged in PR Dashboard Oct 7, 2025
@Quetzacoalt91 Quetzacoalt91 deleted the add-missing-changes-for-ps-9.0.1 branch October 7, 2025 14:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Type: Improvement QA ✔️ Status: Check done, Code approved

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants