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

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 .
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from hasienda. Next status will be 'closed'.
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.