Skip to content

Comments

Update test checking a shop is already installed, because settings.inc.php is not created anymore on PS v9#416

Merged
jolelievre merged 2 commits intoPrestaShop:masterfrom
Quetzacoalt91:update-reference-file-for-installed-shops
Feb 28, 2025
Merged

Update test checking a shop is already installed, because settings.inc.php is not created anymore on PS v9#416
jolelievre merged 2 commits intoPrestaShop:masterfrom
Quetzacoalt91:update-reference-file-for-installed-shops

Conversation

@Quetzacoalt91
Copy link
Member

@Quetzacoalt91 Quetzacoalt91 commented Feb 27, 2025

Questions Answers
Description? config/settings.inc.php is not created anymore on PS v9+. In consequence, the tag nightly will reinstall PrestaShop at each startup because it will wrongly think the shop is not installed.
Type? bug fix
BC breaks? Nope
Deprecations? Nope
Fixed ticket? Related to PrestaShop/PrestaShop#37418
Sponsor company /
How to test? Install a shop with the tag nightly. When you restart it, the container should not reinstall PrestaShop.

@github-project-automation github-project-automation bot moved this to Ready for review in PR Dashboard Feb 27, 2025
@Quetzacoalt91 Quetzacoalt91 changed the title Check new file to get status of shop as settings.inc.php is not creqt… Check new file to get status of shop as settings.inc.php is not created anymore on PS v9 Feb 27, 2025
@Quetzacoalt91 Quetzacoalt91 changed the title Check new file to get status of shop as settings.inc.php is not created anymore on PS v9 Update test checking a shop is already installed, because settings.inc.php is not created anymore on PS v9 Feb 27, 2025
@ps-jarvis ps-jarvis moved this from Ready for review to To be tested in PR Dashboard Feb 28, 2025
@jolelievre jolelievre merged commit 9594167 into PrestaShop:master Feb 28, 2025
2 checks passed
@github-project-automation github-project-automation bot moved this from To be tested to Merged in PR Dashboard Feb 28, 2025
@ps-jarvis ps-jarvis moved this from Merged to To be tested in PR Dashboard Feb 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants