Roundup Tracker - Issues

Issue 2550801

classification
MESSAGES_TO_AUTHOR 'nosy' not on config.ini
Type: resource usage Severity: normal
Components: Mail interface Versions: 1.4
process
Status: closed fixed
:
: pcaulagi : admin, ber, pcaulagi, schlatterbeck, tekberg
Priority: :

Created on 2013-03-20 17:33 by tekberg, last changed 2013-05-12 19:56 by tekberg.

Messages
msg4824 Author: [hidden] (tekberg) Date: 2013-03-20 17:33
There are 2 config.ini files that come with release 1.4.21:
  $ find . -name config.ini
  ./share/roundup/templates/devel/config.ini
  ./share/roundup/templates/responsive/config.ini
The [nosy] section for both of these files contains the 
messages_to_author attribute. The comments above the attribute describe 
the yes, no and new settings. The comment should add a description for 
the nosy setting too. Here is what is there now:

# Send nosy messages to the author of the message.
# Allowed values: yes, no, new
# Default: no

Please consider changing the messages_to_author comment (both files) to 
this:

# Send nosy messages to the author of the message.
# Allowed values: yes, no, new, nosy
#   yes - always
#   no - never
#   new - only when the issue is created
#   nosy - only when the author is in the nosy list
# Default: no

I stumbled on the nosy option when I was comparing old code (1.4.8) for 
roundup/roundupdb.py IssueClass.nosymessage to the new code. I made a 
fix in that old version so that if one replies to a nosy email it goes 
to the sender, not the tracker. It looks like it still works that way.
msg4888 Author: [hidden] (pcaulagi) Date: 2013-05-11 15:56
Thanks for the suggestion.  I have committed the changes here - 
http://sourceforge.net/p/roundup/code/ci/8aa3175fb6ae.
msg4891 Author: [hidden] (tekberg) Date: 2013-05-12 19:56
Looks great! Thank you.
History
Date User Action Args
2013-05-12 19:56:45tekbergsetmessages: + msg4891
2013-05-11 15:56:37pcaulagisetstatus: new -> closed
assignee: pcaulagi
resolution: fixed
messages: + msg4888
nosy: + pcaulagi
2013-03-21 08:16:18schlatterbecksetnosy: + schlatterbeck
2013-03-20 17:33:13tekbergcreate