Skip to content

fix: deactivate custom module 80#1539

Merged
ga-devfront merged 1 commit intoPrestaShop:devfrom
ga-devfront:fix/issue-999
Nov 27, 2025
Merged

fix: deactivate custom module 80#1539
ga-devfront merged 1 commit intoPrestaShop:devfrom
ga-devfront:fix/issue-999

Conversation

@ga-devfront
Copy link
Contributor

@ga-devfront ga-devfront commented Nov 26, 2025

Questions Answers
Description? Issue if I have deactivate non nativ module option checked and I haven't non nativ module on my shop.
Type? bug fix
BC breaks? no
Deprecations? no
Fixed ticket? Internal ticket
Sponsor company @PrestaShopCorp
How to test? see internal ticket

@ga-devfront ga-devfront added this to the 7.5.0 milestone Nov 26, 2025
@ga-devfront ga-devfront added the bug Type: Bug fix label Nov 26, 2025
@github-project-automation github-project-automation bot moved this to Ready for review in PR Dashboard Nov 26, 2025
@sonarqubecloud
Copy link

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 @ga-devfront,

I tested ur pr in the same conditions as the bug was detected (details on the intern ticket) and works ! The upgrade does not fail anymore.

Thanks, it's QA approved ✅

@ingridusta ingridusta added QA ✔️ Status: Check done, Code approved and removed waiting for QA labels Nov 26, 2025
@ga-devfront ga-devfront merged commit 481e848 into PrestaShop:dev Nov 27, 2025
56 checks passed
@github-project-automation github-project-automation bot moved this from Ready for review to Merged in PR Dashboard Nov 27, 2025
@ps-jarvis ps-jarvis moved this from Merged to Ready for review in PR Dashboard Nov 27, 2025
@ga-devfront ga-devfront deleted the fix/issue-999 branch November 27, 2025 07:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Type: Bug fix Migration script QA ✔️ Status: Check done, Code approved

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants