Vote #73232
未完了"Only for things I watch or I'm involved in" sends notifications only for issues
0%
説明
The e-mail notifications setting "Only for things I watch or I'm involved in" currently only sends notifications for issue updates. This has confused some of our (Plan.io) users because they didn't receive notifications for other things than issues they'd watch.
Patch is upcoming.
journals
Patch is attached.
--------------------------------------------------------------------------------
Hi Felix,
just to make sure I understand the scenario correctly:
# User 1 sets watch=yes on something else than an issue, say, a wiki page
# User 2 changes that wiki page
# User 1 does not get a mail
I just tested this on 2.2.1 and I got a mail...
Edit: "Only for things I watch or I'm involved in" being my preference, of course.
--------------------------------------------------------------------------------
In your scenario, if user 1 has the email notification setting on "Only for things I watch or I'm involved in", he won't get an email. The german translation is a little misleading because it says "Only for issues I watch or I'm involved in", the french translation is also misleading because it says "Only for things I watch".
--------------------------------------------------------------------------------
Felix Schäfer wrote:
> The german translation is a little misleading because it says "Only for issues I watch or I'm involved in",
It reads 'Aufgaben'... I agree, that should be corrected...
> the french translation is also misleading because it says "Only for things I watch".
Just like the french version, I'll include that.
> In your scenario, if user 1 has the email notification setting on "Only for things I watch or I'm involved in", he won't get an email.
I checked that @work this afternoon and well, I got a mail...
I just tested at home and, alas, no mail.
The difference being that @work my user is an administrator...
I'll re-check tomorrow @work.
--------------------------------------------------------------------------------
The current implementation states that it's only about Issues so changing it require an update of this statement, all the different case to be dealt with and some unit tests for "things" other than issues.
--------------------------------------------------------------------------------
Etienne Massip wrote:
> The current implementation states that it's only about Issues so changing it require an update of this statement, all the different case to be dealt with and some unit tests for "things" other than issues.
The statement is in the code, yes, but the email notifications options don't reflect that (except in the german translation).
I first wanted to hear the consensus about this issue before fixing everything around it. If you say this is OK for core I'll happily update the patch to correct the comment, tests and german, french and english translations. If you think this shouldn't go into Redmine, I obviously don't need to do the work :-)
--------------------------------------------------------------------------------
Any news on this?
--------------------------------------------------------------------------------
related_issues
relates,New,8406,Missing notification "Only for things I watch or I'm assigned to" for developers
relates,New,13012,DE and FR: translation wrong for label_user_mail_option_only_my_events