Modify

Opened 4 years ago

Closed 4 years ago

#7494 closed defect (fixed)

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

Reported by: JirkaV Owned by: hasienda
Priority: normal Component: AnnouncerPlugin
Severity: normal Keywords: email_enabled fullblog documentation
Cc: hasienda Trac Release: 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

Attachments (0)

Change History (3)

comment:1 Changed 4 years ago by rjollos

  • Cc hasienda added; anonymous removed

comment:2 Changed 4 years ago by hasienda

  • Owner changed from doki_pen to hasienda
  • Status changed from new to assigned
  • Summary changed from email_enabled should be set to True or documented to '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 4 years ago by hasienda

  • Keywords documentation added
  • Resolution set to fixed
  • Status changed from assigned to 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.

Add Comment

Modify Ticket

Action
as closed The owner will remain hasienda.
The resolution will be deleted. Next status will be 'reopened'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.