'email_enabled' should default to 'True' or better documented

[Jirka Vejrazka]

I'm using trunk version of AnnouncerPlugin as I need the included FullBlog notification.

I found out that it was not working well for me. A little bit of digging revealed that no emails were being sent due to email_enabled = false (the default).

As I could not find this being documentes, I guess it's a bug. Either email_enabled should default to True (it is a notification plugin anyway :) or the need to set email_enabled in trac.ini should be documented.

Many thanks for providinig this awesome plugin!

Jirka

[Steffen Hoffmann]

I think, you're right. The closest is the hint on doing like a setup for ​TracNotification in the configuration section of AnnouncerPlugin, but this would just lead to

[notification]
smtp_enabled = true

not

email-enabled

Currently you'll better read the source docu, but as said before, I agree, that there should be better, (user-)friendly documentation, even if AnnouncerPlugin is somewhat involving, so you're bound to spend some time with it anyway.

I'll try to figure out about a proper wording for plugin's home here. But hey, why not help with this yourself as well? After all it's a Wiki, right? :-)

[Steffen Hoffmann]

After at least three other tickets and no one else acting on the matter I took my time for a major revision of AnnouncerPlugin's wiki page.

I fell much more comfortable now, and I hope that you'll agree with me. If not, feel free to again/further raise your critics here - politely.