Redmine

I'm sorry, I missed one change. Here is the tiny additional
change.

The only thing I would add is that it is nice to be able to sometimes
include people who may not be aware that they should be included.
They may not even be RedMine users. In bugzilla I would regularly
CC managers or ever sales people so that they could keep up to
date on an issue. Just a suggestion.

Oh, but that is already included - everybody, who is
autorized to add members to the project can add or remove
other watchers - just scroll to the bottom of the "show
issue" page!

I didn't want everybody to add watchers - but if I can
change my mind :-)

muro

I've just added watching functionality.
For now, it's not possible:
- to add someone else
- to view the list of watchers
It will come asap.

I used a different approach (dedicated model and controller)
to allow other things to be watched in the future (eg. forums,
document, wiki page...).

Thanks, you are right - it is certainly better to allow watching
of forums,
documents etc.

I really like the icon, would it be possible to have a holow
one for one of the
states (e.g. hollow star = issue is not being watched)

Thanks a lot, muro

I made a grey scale icon. Hope you'll like :-)

sounds good! I'll take a look when it's commited into svn (hint,
hint) :-)

muro

It took me some time to understand :-)

Sorry for the cryptic comment :-). Thanks, I like the new icon.
And thanks for the
commited code - I'm not a rails nor ruby expert and it's
an interesting solution.
So I learned something new today. Thanks, Jean Philippe.

muro

Oh, I just found one more problem - but this might be my
misunderstanding - I would like to let a user know, that an
issue was just assigned to him. I did this in
app/models/mailer.rb, after line 34:

@recipients.push  issue.assigned_to.mail if
issue.assigned_to && !issue.assigned_to.mail.empty?
    @recipients.uniq!

Another thing: I had a check before line 31:
return unless journal
I'm not sure if it is necessary.

In trunk source, all of the project members are receiving the
notifications. So adding the assign user is not necessary.
I know it could be better. I'll try to make mail notifications
more configurable for people, so they can choose what kind of
notifications they receive.

For the second point, did you ever get a nil error
in Mailer#issue_edit ?

I understand your point. However - we already have hundreds
of issues and getting a mail notification for each change
too much. Most people have mail notifications turned off
(except for one or two people) and only want to receive
notifications if something changes with an issue assigned to
them. That's the reason for the proposed change.

regarding the nil error: you are right - it is probably just
a leftover from development while I was implementing the
feature.

Thanks, muro