Since Mail mime has not been ported yet, we need to port this class to HTML mail.

Comments

emartoni created an issue. See original summary.

emartoni’s picture

Assigned: Unassigned » emartoni
emartoni’s picture

Assigned: emartoni » Unassigned
Status: Active » Needs review
StatusFileSize
new40.91 KB
emartoni’s picture

emartoni’s picture

renatog’s picture

Status: Needs review » Needs work
Issue tags: +Needs reroll

Please, re-roll the patch.

Thanks

emartoni’s picture

StatusFileSize
new42.19 KB

Hi @RenatoG.

New patch attached!

Best,
Eduardo

emartoni’s picture

Status: Needs work » Needs review
renatog’s picture

Status: Needs review » Reviewed & tested by the community

works good now

  • RenatoG committed c241ed7 on 8.x-1.x authored by emartoni
    Issue #2953221 by emartoni: Port Mailmime class to HTML Mail module
    
renatog’s picture

Status: Reviewed & tested by the community » Fixed

Committed.

Thank you very much

Utkarsh_Mishra’s picture

Status: Fixed » Needs work

It is still throwing an error as in issue [here](https://www.drupal.org/project/htmlmail/issues/2954454) reverting the changes seems to fix that issue.

emartoni’s picture

Status: Needs work » Needs review
StatusFileSize
new2.21 KB

Hi @Utkarsh_Mishra, thanks for your time!

Unfortunately, a revert is not a solution for that.
Instead, I changed the class used and looks fine.

I added a new patch with code fixed.

Best,
Eduardo

renatog’s picture

Status: Needs review » Reviewed & tested by the community

Hi guys.

I applied the patch and really works good.

Thank you very much for contributions dudes.

Best,

  • RenatoG committed 2f3a523 on 8.x-1.x authored by emartoni
    Issues #2953221 #2954454 by emartoni, RenatoG, Utkarsh_Mishra: Port...
renatog’s picture

Status: Reviewed & tested by the community » Fixed
Issue tags: -Needs reroll

Fixed.

Committed to the dev branch: 8.x-1.x

Thank you very much.

Best,

Status: Fixed » Closed (fixed)

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