I checked the process of test addresses. Finally i've realized that i have missed the information text to understand, if subscription properties / user relationship also get applied to test messages -- or if it's only a trivial test message to the supplied email.

This is great, since it happens, but a comment about this fact in the test form or the test properties would make things much clearer. Keyword Subscription / Account / User missing.

Currently:

admin/settings/simplenews/newsletter
Fieldset: Test addresses
Supply a comma-separated list of email addresses to be used as test addresses. The override function allows to override these addresses in the newsletter editing form.

May be also for node form

node/add/simplenews
Fieldset: Test email addresses:
Supply a comma-separated list of email addresses to be used as test addresses.

Comments

sutharsan’s picture

Version: 7.x-1.x-dev » 6.x-2.x-dev
Status: Active » Fixed

I don't no where it went in but the suggested text is currently present on the node form.

miro_dietiker’s picture

Status: Fixed » Needs work

Hmm.. i see my words wheren't clear enough.

Once you have token support you might ask yourself how to send a testmail that has access to tokens / related account information.
What i'd like to see to understand it, is something like "Simplenews will autoload accounts with identical mail address - not only send anonymous test mails".

sutharsan’s picture

If I put myself in the position of newsletter editor - if that is still possible ;) - I expect the test emails to behave just like real newsletters. So I - as developer- don't see the need to describe the obvious behavior. We might give a warning message when the test newsletter is sent to an address which is not subscribed.

miro_dietiker’s picture

right then ;-)
that's indeed also a great idea. :-)

miro_dietiker’s picture

Status: Needs work » Needs review
StatusFileSize
new1.76 KB

Providing patch for review. Implementing two different messages for anonymous test sends and authenticated test sends.

miro_dietiker’s picture

Version: 6.x-2.x-dev » 7.x-1.x-dev
Status: Needs review » Patch (to be ported)
StatusFileSize
new1.72 KB

Reviewed once again, tested much and committed to dev.
I think this functionality should be ported to D7.

DrupOn’s picture

Status: Patch (to be ported) » Fixed

Committed to dev.

Status: Fixed » Closed (fixed)

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