Problem/Motivation

Noticed that the email notifications for PSAs aren't backported to D7.

Proposed resolution

Add it

Remaining tasks

Backport

User interface changes

API changes

Data model changes

Release notes snippet

CommentFileSizeAuthor
#4 3072800-4.patch8.83 KBheddn
#4 interdiff_3-4.txt2.78 KBheddn
#3 3072800-3.patch6.48 KBheddn
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

heddn created an issue. See original summary.

heddn’s picture

heddn
Primary language English
Location Nicaragua
CreditAttribution: heddn at MTech, LLC, Tag1 Consulting for Drupal Association, European Commission and European Union Institutions, Agencies and Bodies commented
Status: Active » Needs review

Here's the backport. I discovered while backporting a few bugs in D8. I'll open an issue for that.

Namely, we're not honouring the 'notification' toggle in the admin UI and always send notifications anyway.

heddn’s picture

heddn
Primary language English
Location Nicaragua
CreditAttribution: heddn at MTech, LLC, Tag1 Consulting for Drupal Association, European Commission and European Union Institutions, Agencies and Bodies commented
FileSize
3072800-3.patch6.48 KB

Missed uploading the patch. Here it is now.

heddn’s picture

heddn
Primary language English
Location Nicaragua
CreditAttribution: heddn at MTech, LLC, Tag1 Consulting for Drupal Association, European Commission and European Union Institutions, Agencies and Bodies commented
FileSize
interdiff_3-4.txt2.78 KB
3072800-4.patch8.83 KB

  • heddn committed be98ba8 on 7.x-1.x 
    Issue #3072800 by heddn: Backport email notifications
heddn’s picture

heddn
Primary language English
Location Nicaragua
CreditAttribution: heddn at MTech, LLC, Tag1 Consulting for Drupal Association, European Commission and European Union Institutions, Agencies and Bodies commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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