- Issue created by @marioki
- Status changed to Postponed: needs info
over 1 year ago 6:07pm 8 August 2023 - π¬π§United Kingdom adamps
Please provide full "steps to reproduce".
At /admin/config/system/mailer/override you have a choice.
- If you disable the override for commerce order then it should use the subject and template from commerce.
- If you enable it, then it should use the Mailer Policy settings from this module, and you should get sensible defaults.
Please report what state you currently have. You could try to run "reset" or "re-import" and it might fix things.
- π«π·France notFloran Reims
I had the same problem: I had to "enable" the override and then disable it to get everything working properly again
- Status changed to Active
over 1 year ago 7:59pm 8 August 2023 - ππΊHungary Large64
Method written in #3 helped. I enabled all overrides then disabled them, everything went back to normal.
- Status changed to Needs review
over 1 year ago 6:28pm 9 August 2023 - last update
over 1 year ago 6 pass - π¬π§United Kingdom adamps
OK I think I have a patch (also fixes π Fresh installations emails broken with 1.3.0 Fixed ). Please can someone test?
- π¨π·Costa Rica maxmendez
Hi @AdamPS,
I've tested it in my dev commerce environment and seems that the problem is fixed. Thanks for your work.
- Status changed to RTBC
over 1 year ago 2:22pm 10 August 2023 -
AdamPS β
committed 325d06c6 on 1.x
Issue #3379690 by AdamPS, notFloran, MaxMendez: Update from 1.2 to 1.3...
-
AdamPS β
committed 325d06c6 on 1.x
- Status changed to Fixed
over 1 year ago 2:26pm 10 August 2023 - πΊπΈUnited States morbus iff
For existing sites, this new functionality is... crazy.
Whether the DSM override is enabled or not, the DSM module takes over the Commerce Order types UI (admin/commerce/config/order-types/*/edit), so that an admin can no longer set Commerce's native BCC and Subject line fields. When it IS enabled, it seems to create multiple policies (a default one for ALL/ALL, then an imported one for Receipt/orderTypeID) and NEITHER of those show up in the Commerce Order types UI.
When the DSM override is enabled, the "sensible Body default" seems to use the "current user's permissions" to send out the email, such that an admin testing an order will see *admin-only comments* in the email AND see form controls for adding comments (as an admin would normally see on admin/commerce/orders/#).
And I've yet to get "have the DSM override enabled, but allow continued use of Commerce's default twig template for the Body". If I delete the Body override element entirely, it somehow still uses the "sensible Body default", sending out the bad unstyled HTML (caused by another issue: Warning: file_get_contents(default/files/css/css_I5WrM2XNc4EO21P353X1Tp3NJ4d0oh87oqBOCBGEP18.css?delta=0&language=en&theme=edv_bootstrap&include=eJxLTSmLT8rPLykuKUos0E_NTczM0SmuzE3Lz6uMB3FSi_ST83NzU4uSU-Pzi1JSiwDuVRPD): Failed to open stream: No such file or directory in Drupal\symfony_mailer\Plugin\EmailAdjuster\InlineCssEmailAdjuster->postRender() (line 78 of modules/contrib/symfony_mailer/src/Plugin/EmailAdjuster/InlineCssEmailAdjuster.php). ) with admin comments.
- π¬π§United Kingdom adamps
@Morbus Iff Please raise a new issue - what you see seems very strange
And I've yet to get "have the DSM override enabled, but allow continued use of Commerce's default twig template for the Body" to work. Is it supposed to?
No
Automatically closed - issue fixed for 2 weeks with no activity.