Problem/Motivation

Currently, if you send a test mail, "A test e-mail has been sent ..." confirmation message is shown, but colored in a yellow warning color, with exclamation mark and the header "Warning message".

Also, if you enable "Enable Postmark debugging to not send an email", the "Email successfully tested, no email has been sent" message is shown as a warning, even though it was a success.

Steps to reproduce

Send a test email, or enable "Enable Postmark debugging", and get a warning message when sending mail.

Proposed resolution

Show the messages as a success, since an email was successfully sent, or an email was prevented from being sent, since both cases it's a success.

Possibly, the "Email successfully tested, no email has been sent" should stay as a warning, since it might not be what the user wants for the production web site? Still, it is only used on the form page itself, and as a confirmation ...

Postmark message sent

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#6 postmark-success.png78.19 KBressa

Issue fork postmark-3390761

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

ressa created an issue. See original summary.

ressa’s picture

Assigned: ressa » Unassigned
Category: Bug report » Feature request
Status: Active » Needs review
ressa’s picture

Issue summary: View changes
ressa’s picture

Title: Show test mail sent confirmation message as success » Show test mail sent and "no email sent" (debug) as success messages
ressa’s picture

Issue summary: View changes
StatusFileSize
new78.19 KB
ressa’s picture

Issue summary: View changes
i-trokhanenko’s picture

Status: Needs review » Reviewed & tested by the community

  • i-trokhanenko committed ea8c20a5 on 8.x-1.x authored by ressa
    Issue #3390761: Show test mail sent confirmation message as success
    
i-trokhanenko’s picture

Status: Reviewed & tested by the community » Fixed

Thanks!

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

ressa’s picture

Thanks for maintaining the module @i-trokhanenko!