🇬🇧United Kingdom @PQ

So I seem to have managed to unintentionally create a fork. Don't know if that's undoable.

Really sorry about that!

PQ → made their first commit to this issue’s fork.

Just a note that the current solution here seems to break when used with the sendgrid_integration module which validates the email addresses of the bcc and cc headers. If empty it throws an error.

It might be resolved by changing this:

      $data['to'] = implode(',', $recipients['to']);
      $data['params']['headers']['Cc'] = implode(',', $recipients['cc']);
      $data['params']['headers']['Bcc'] = implode(',', $recipients['bcc']);

(in src/Notification.php line ~213) to something like:

      $data['to'] = implode(',', $recipients['to']);
      if (!empty($recipients['cc'])) {
        $data['params']['headers']['Cc'] = implode(',', $recipients['cc']);
      }
      if (!empty($recipients['bcc'])) {
        $data['params']['headers']['Bcc'] = implode(',', $recipients['bcc']);
      }

Appears to work fine, any chance of a commit (if only into dev) so it can play nicely with composer on D10 sites?