In preparation of #3390319: Implement a queue mechanism, the code that sends emails, which will be used by two different plugins, must be moved to a service that will be used from both the plugins.

This issue is just to move code from the mail plugin to a service class that can be used from two different type of plugins the module will use.

Comments

apaderno created an issue. See original summary.

avpaderno’s picture

Assigned: Unassigned » avpaderno
avpaderno’s picture

Issue summary: View changes

Any other change like #3390319: Implement a queue mechanism must wait this issue is fixed. I am working on this and already changing the module code (adding two new classes).

The changes I am introducing are slightly different from the changes introduced from the current patch in #3390319: Implement a queue mechanism.

avpaderno’s picture

I will commit the code between 3 hours, after verifying the changes are correct and the BC layer is in place.

apaderno credited cgoffin.

avpaderno’s picture

  • apaderno committed 4c3ebc42 on 2.0.x
    Issue #3392607 by apaderno, cgoffin, dave.mentens: Move the code that...
avpaderno’s picture

I would like to understand how much sites using this module are still on Drupal 9, and when they will upgrade to Drupal 10. This would be information useful to decide about back-ports, although the module has never had a stable release. (That means the project API could still be changed and the back-ports are still possible.)

avpaderno’s picture

Status: Active » Fixed

Status: Fixed » Closed (fixed)

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