Edgewall Software
Modify

Opened 18 years ago

Closed 17 years ago

#3120 closed enhancement (duplicate)

[PATCH] cc field in tickets should be remove if smtp isn't enabled.

Reported by: anonymous Owned by: Jonas Borgström
Priority: normal Milestone:
Component: ticket system Version: 0.9.5
Severity: minor Keywords: notification email
Cc: tbrkic@… Branch:
Release Notes:
API Changes:
Internal Changes:

Description

Hi,

the cc field is visible even if the trac installation isnt configured to send emails.

I think it should be removed. Or maybe greyed out.. so that trac-admin knows that it could be enabled. Maybe with a link to Trac Documentation on how to setup email sending.

/T

Attachments (1)

trac-#3120.patch (4.5 KB ) - added by Emmanuel Blot 18 years ago.
Patch against the trunk

Download all attachments as: .zip

Change History (7)

comment:1 by Emmanuel Blot, 18 years ago

Keywords: notification email added
Severity: normalminor
Type: defectenhancement

comment:2 by Christopher Lenz, 18 years ago

I don't completely agree. The CC field can be used as a watch list through custom queries even when SMTP notifications are not enabled.

OTOH, I do think that we ultimately need to generalize/redesign the CC field (see #1459).

comment:3 by Emmanuel Blot, 18 years ago

Attaching a patch for the trunk [3271].

I don't think it is worth merging it to the trunk.

by Emmanuel Blot, 18 years ago

Attachment: trac-#3120.patch added

Patch against the trunk

comment:4 by anonymous, 18 years ago

I never occured to me that you could using it for custom queuries. Which is quite nice. But I think the reason that it can be used for things it is not meant for is a little wrong reason for not changing it. Then I would rather add another field that is labeled custom query tags or something. So that it becomes obvious and as a new user you also realize that you can do stuff like that.

The reason for this ticket is actually that trac is becoming popular and more and more open source project use it. Which is good. BUT I dont like to enter my email address due to spam, and it is impossible for me to know when I enter the ticket if the cc sending works or not.

So my vote is vote is to merge the patch /T

comment:5 by sid, 18 years ago

Summary: cc field in tickets should be remove if smtp isn't enabled.[PATCH] cc field in tickets should be remove if smtp isn't enabled.

comment:6 by Christian Boos, 17 years ago

Resolution: duplicate
Status: newclosed

The Cc: field is not necessarily tied to the SMTP settings: it might well be that in the future we'll have other ways for notifying than just SMTP (#3517). There will still presumably be only one CC: field (or the equivalent nosy list, see #1459), and the choice of the notification medium could be done in the user preference (#4056).

If nevertheless you think the Cc: field should be hidden in your situation, then it's a similar matter than #3281.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Jonas Borgström.
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from Jonas Borgström to the specified user.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.