Skip to content

Commit

Permalink
fix: magento2 requires mariadb 10.6, for #5919 (#5922)
Browse files Browse the repository at this point in the history
  • Loading branch information
rfay committed Mar 4, 2024
1 parent e4b45d5 commit e294282
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
2 changes: 1 addition & 1 deletion docs/content/users/quickstart.md
Original file line number Diff line number Diff line change
Expand Up @@ -363,7 +363,7 @@ The Laravel project type can be used for [Lumen](https://lumen.laravel.com/) lik

```bash
mkdir ddev-magento2 && cd ddev-magento2
ddev config --project-type=magento2 --php-version=8.1 --docroot=pub --disable-settings-management
ddev config --project-type=magento2 --php-version=8.1 --database=mariadb:10.6 --docroot=pub --disable-settings-management
ddev get ddev/ddev-elasticsearch
ddev start
ddev composer create --repository=https://repo.magento.com/ magento/project-community-edition -y
Expand Down
4 changes: 3 additions & 1 deletion pkg/ddevapp/magento.go
Original file line number Diff line number Diff line change
Expand Up @@ -165,8 +165,10 @@ func magentoConfigOverrideAction(app *DdevApp) error {
return nil
}

// Latest magento2 requires php8.1
// Magento2 2.4.6 requires php8.1/2 and MariaDB 10.6
// https://experienceleague.adobe.com/docs/commerce-operations/installation-guide/system-requirements.html
func magento2ConfigOverrideAction(app *DdevApp) error {
app.PHPVersion = nodeps.PHP81
app.Database = DatabaseDesc{Type: nodeps.MariaDB, Version: nodeps.MariaDB106}
return nil
}

0 comments on commit e294282

Please sign in to comment.