Problem/Motivation

The Graph Mail formatter is expected to convert the message body to a single string per \Drupal\Core\Mail\MailInterface::format(), see https://git.drupalcode.org/project/drupal/-/blob/11.x/core/lib/Drupal/Co...

Proposed resolution

Convert the message body to a single string in \Drupal\graph_mail\Plugin\Mail\GraphMail::format()

Remaining tasks

  1. Write a patch
  2. Review
  3. Commit

User interface changes

None.

API changes

None.

Data model changes

None.

Issue fork graph_mail-3422609

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

idebr created an issue. See original summary.

idebr’s picture

idebr commented
Status: Active » Needs review

In the merge request the message body is converted to a single string in \Drupal\graph_mail\Plugin\Mail\GraphMail::format()

idebr opened merge request !19

bramdriesen’s picture

bramdriesen
he/him
Primary language Dutch
Location Belgium 🇧🇪🇪🇺
commented
Status: Needs review » Needs work

Thanks for your contribution once again @idebr! I added one comment.

bramdriesen’s picture

bramdriesen
he/him
Primary language Dutch
Location Belgium 🇧🇪🇪🇺
commented
Status: Needs work » Reviewed & tested by the community

Comment is answered ;-)

  • BramDriesen committed 584e6aba on 2.0.x authored by idebr 
    Issue #3422609 by idebr: \Drupal\graph_mail\Plugin\Mail\GraphMail::...
bramdriesen’s picture

bramdriesen
he/him
Primary language Dutch
Location Belgium 🇧🇪🇪🇺
commented
Status: Reviewed & tested by the community » Fixed
bramdriesen’s picture

bramdriesen
he/him
Primary language Dutch
Location Belgium 🇧🇪🇪🇺
commented

Also tagged a new release: https://www.drupal.org/project/graph_mail/releases/2.0.0-alpha5

  • apaderno committed 3d927714 on 1.0.x 
    Issue #3422609: \Drupal\graph_mail\Plugin\Mail\GraphMail::format()...

Status: Fixed » Closed (fixed)

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

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Version: 2.0.x-dev » 1.0.x-dev