We should add the ability to test an email configuration via a "Test" operation on the email. It should present a form that asks you to specify the email address to send a test copy of the email to and which order should be used to provide the context for token replacement. Conditions should be ignored, and the test operation should not be logged to the order's log. It should prepend "[TEST]" to the subject line.

CommentFileSizeAuthor
#2 3144974-test-email-1.patch7 KBTylerMarshall

Comments

rszrama created an issue. See original summary.

TylerMarshall’s picture

TylerMarshall commented
Assigned: Unassigned » TylerMarshall
Status: Active » Needs review
StatusFileSize
new 3144974-test-email-1.patch7 KB

This adds a link in the operations, and on the edit page (once an email has been created) that copy pastes the email fields with a few additions, and provides an action to send test it with the order context.

daggerhart’s picture

daggerhart commented
Status: Needs review » Reviewed & tested by the community

This works as described.

loze’s picture

loze commented

Looks good to me.
+1 Thanks

rszrama’s picture

rszrama commented

Thanks for the prior reviews! On my local test I noticed this was hard coded to ask for an order reference in the test form, but the entity type is actually variable. I've updated the patch so the test email form adapts to whatever entity type the email targets. I've also added a confirmation message on submission.

  • rszrama committed 47361c9 on 8.x-1.x authored by TylerMarshall 
    Issue #3144974 by TylerMarshall, rszrama, daggerhart, loze: Add a "Test...
rszrama’s picture

rszrama commented
Assigned: TylerMarshall » Unassigned
Status: Reviewed & tested by the community » Fixed

Committed.

Status: Fixed » Closed (fixed)

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