Problem/Motivation

There's this handy mail templating / editing tool Mail Editor. It allows you to manage all the mails (templates) in one place, further it provides multi-language support and extended tokens (logic operations, loops).

Proposed resolution

The attached patch contains all the necessary stuff to integrate the Mime Mail rules actions with Mail Editor. However, the patch currently relies on Mail Editor changes made in: #212224: Mime Mail module: HTML emails and Plain text emails
The attached patch relies on the latest patch in the related issue.

Remaining tasks

Reviews needed

User interface changes

none

API changes

none

Has to be fixed first: #212224: Mime Mail module: HTML emails and Plain text emails

CommentFileSizeAuthor
#3 mimemail-add-mail_edit-support-2033637-3.patch7.36 KBdas-peter
#3 interdiff-2033637-2-3-do-not-test.diff1.1 KBdas-peter
#2 mimemail-add-mail_edit-support-2033637-2.patch6.9 KBdas-peter
#2 interdiff-2033637-0-2-do-not-test.diff2.57 KBdas-peter
mimemail-add-mail_edit-support.patch7.02 KBdas-peter

Comments

sgabe’s picture

sgabe commented
Status: Needs review » Postponed

Thanks for your work das-peter, but since this issue is blocked by #212224: Mime Mail module: HTML emails and Plain text emails I am marking this as postponed for now. I seriously hope there will be a solution and salvis will support this pursuit.

das-peter’s picture

das-peter commented
Issue summary: View changes
Related issues: +#2076827: Add plaintext field, +#212224: Mime Mail module: HTML emails and Plain text emails
StatusFileSize
new interdiff-2033637-0-2-do-not-test.diff2.57 KB
new mimemail-add-mail_edit-support-2033637-2.patch6.9 KB

Updated patch to integrate with #2076827: Add plaintext field
@sgabe I absolutely see the point for postponing this - but I still hope I can get some movement into this asap.

das-peter’s picture

das-peter commented
StatusFileSize
new interdiff-2033637-2-3-do-not-test.diff1.1 KB
new mimemail-add-mail_edit-support-2033637-3.patch7.36 KB

Changed function to load rules. The previous function call missed reaction rules.
Btw. I've no clue why there's an error with the patch version.

tr’s picture

tr commented
Status: Postponed » Closed (won't fix)

This would be a lot of work to do, with very little benefit at this point. I'm marking this as "Closed won't fix" because Drupal 7 is nearing its end of life, because the community hasn't contributed anything to this feature request for almost eight years, and because I have no interest in doing this myself.