Closed (fixed)
Project:
SendGrid Integration
Version:
8.x-2.x-dev
Component:
Code
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
6 Sep 2021 at 12:58 UTC
Updated:
13 Feb 2023 at 10:14 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
kevinhbruce commentedI'm running into the same issue with Drupal Commerce's use of sendgrid integration
Comment #3
kevinhbruce commentedI was able to quiet the message by taking that code and replacing it with the following:
Comment #4
anas_maw commentedRunning to the same problem, code in #3 solved my issue
I just make a patch for the code in #3
Comment #5
intrafusionThanks for the patch, but you cannot review your own patch
Comment #6
anas_maw commentedThanks for your note, I reviewed the code in #3, and just created a patch for it.
Anyway, please review it and move it to RTCB as this will cause the whole email to fail if there is a CC or BCC with a capital case
Comment #7
metafoo commentedPatch #4 is broken.
Fixes error getting flagged, but functionality is still broken.
If item contains both cc and bcc, the cc is clobbered by re-initialising array $address_cc_bcc, thus will not send the cc.
Removing the line
fixes. Just let the array get initialised when its first used.
Comment #10
parisekCreated MR from attached patch and fixed #7
Changed to major as if both CC and BCC recepients are set, only later one is sent.
Comment #11
perignon commentedPR merged. Thanks!
Comment #13
intrafusion