Opened 7 years ago

Closed 7 years ago

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

Reported by: Owned by: Jirka Vejrazka Steffen Hoffmann normal AnnouncerPlugin normal email_enabled fullblog documentation Steffen Hoffmann 0.11

### Description

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

### comment:1 Changed 7 years ago by Ryan J Ollos

Cc: Steffen Hoffmann added; anonymous removed

### comment:2 Changed 7 years ago by Steffen Hoffmann

Owner: changed from Robert Corsaro to Steffen Hoffmann new → assigned email_enabled should be set to True or documented → 'email_enabled' should default to 'True' or better documented

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? :-)

### comment:3 Changed 7 years ago by Steffen Hoffmann

Keywords: documentation added → fixed assigned → closed

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.

### Modify Ticket

Change Properties