Ludwig integration is the Composer installation alternative.

CommentFileSizeAuthor
#2 3256190-02.patch1 KBdevad

Comments

devad created an issue. See original summary.

devad’s picture

Status: Active » Needs review
StatusFileSize
new1 KB

This should do the job. The Symfony Mailer module should work just nice.

The dummy 'exclude-from-classmap' Ludwig notices can be safely ignored.

The Symfony Mailer module is destined to be the main Drupal mailer module in years to come. It would be nice to have this committed as a support for all Drupal users who are not familiar with Composer yet - or not using it in their Drupal project for other reasons.

adamps’s picture

Hi @devad thanks for the patch.

I'm happy to add this feature. However how does it get maintained?

  1. The dependencies of this module may change
  2. There are exact versions specified and these will gradually become outdated, or even insecure

I only use composer so I wouldn't really be able to maintain it myself.

devad’s picture

Thank you for reply @AdamPS.

The Ludwig module has about 10k users. Some of them very active to keep Ludwig integrations in contrib modules up to date.

If it is ok for you to commit community ludwig.json file patches when library versions become seriously outdated or unsecure... that would be enough. Usually, the ludwig.json file needs an update when the composer.json file is updated... which is not so often.

If you need... you can read more at Ludwig FAQ section here.

  • AdamPS committed 7d1c733 on 1.x authored by devad
    Issue #3256190 by devad, AdamPS: Symfony Mailer - Add Ludwig integration
    
adamps’s picture

Status: Needs review » Fixed

OK that makes sense thanks for explaining

devad’s picture

Great. Thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.