- 🇸🇰Slovakia poker10
Thanks for reporting and working on this.
It seems like the D7 code is the same as the D10 code regarding the indentation (see MailFormatHelper::htmlToText() vs drupal_html_to_text()), so I think this needs to be evaluated/fixed in D10 first. Moving it there. We would need to create an MR for 11.x-dev (probably based on the D7 patch). Thanks!
- Status changed to Needs review
8 months ago 12:32pm 12 April 2024 - Status changed to Needs work
8 months ago 12:45pm 12 April 2024 The Needs Review Queue Bot → tested this issue.
While you are making the above changes, we recommend that you convert this patch to a merge request → . Merge requests are preferred over patches. Be sure to hide the old patch files as well. (Converting an issue to a merge request without other contributions to the issue will not receive credit.)
- First commit to issue fork.
- Status changed to Needs review
8 months ago 1:19pm 12 April 2024 - Status changed to Needs work
7 months ago 5:56pm 13 April 2024 - 🇺🇸United States smustgrave
Issue summary should be updated to use the standard issue template
- Status changed to Needs review
7 months ago 6:36pm 13 April 2024 - 🇮🇳India keshav patel
Added the issue summary according to the standard issue template.
- 🇺🇸United States smustgrave
Hiding patch files.
Test-only https://git.drupalcode.org/issue/drupal-3021619/-/jobs/1312508 didn't give a failure like expected so leaving in review.
- Status changed to RTBC
7 months ago 11:00pm 19 April 2024 - 🇺🇸United States smustgrave
Was able to get a test-only feature https://git.drupalcode.org/issue/drupal-3021619/-/jobs/1378481
Seems like a small enough change.
- Status changed to Needs work
7 months ago 9:17am 21 April 2024 - 🇬🇧United Kingdom alexpott 🇪🇺🌍
This seems fine. I think we should issue a change record for this. Can be set back to RTBC once it exists.