ddev icon indicating copy to clipboard operation
ddev copied to clipboard

Add Symfony Mailer config for Mailhog in Drupal 9 settings #2706

Open luenemam opened this issue 3 years ago • 1 comments

The Problem/Issue/Bug:

How this PR Solves The Problem:

Manual Testing Instructions:

mkdir my-drupal9-site
cd my-drupal9-site
ddev config --project-type=drupal9 --docroot=web --create-docroot
ddev start
ddev composer create "drupal/recommended-project" --no-install
ddev composer require drush/drush --no-install
ddev composer require drupal/symfony_mailer --no-install
ddev composer install
ddev drush site:install -y
ddev drush pm:install symfony_mailer
ddev drush user:login "admin/config/system/mailer/test"

Use the login link to go to the page send a test mail. Verify mailhog recieves the test mail.

Automated Testing Overview:

Related Issue Link(s):

#2706

Release/Deployment notes:

luenemam avatar Sep 19 '22 10:09 luenemam

Let me know when you've manually tested and like it, and set it to be ready for review. Thanks for this!

rfay avatar Sep 25 '22 16:09 rfay

A minor update to the docs could be added to this, with a link to symfony mailer

rfay avatar Sep 26 '22 16:09 rfay

This might make it into next point release if it happens this week :)

rfay avatar Oct 04 '22 14:10 rfay

Rebased to pick up docs changes and a number of other things. Make sure you update your local appropriately.

rfay avatar Oct 05 '22 17:10 rfay

I've tested on GitPod. It's working as expected.

luenemam avatar Oct 07 '22 08:10 luenemam

Thanks, I think adding the drupal10 config pre-emptively causes no harm and leaves out a whole future maintenance step.

rfay avatar Oct 07 '22 14:10 rfay

Testing. Can't believe symfony mailer is alpha.

rfay avatar Oct 07 '22 16:10 rfay

Yes, they are ambitious to design a new mail system for drupal.

luenemam avatar Oct 07 '22 16:10 luenemam