diff options
author | Sean McGivern <sean@gitlab.com> | 2017-12-01 15:47:28 +0000 |
---|---|---|
committer | Sean McGivern <sean@gitlab.com> | 2017-12-04 09:25:39 +0000 |
commit | 4b1dd6c02fc2fe2b66a95a05d39c709a4f2922c8 (patch) | |
tree | 7b06152657fdc65157802dd00dc48b57875f5557 /changelogs/unreleased/38862-email-notifications-not-sent-as-expected.yml | |
parent | e0f84130567dc34edf1ae75fcf595e24991d2fa9 (diff) | |
download | gitlab-ce-38862-email-notifications-not-sent-as-expected.tar.gz |
Fix watch level for mentions in description38862-email-notifications-not-sent-as-expected
For a user with the mention notification level set, the type of their
corresponding NotificationRecipient must be :mention for them to receive an
email.
We set this correctly on notes, but we weren't adding it on new issues or MRs -
perhaps because these users are also participants. But the type of the
NotificationRecipient in that case would be :participant, not mention, so we
have to add the mentioned users manually when creating an issue or MR.
When editing an issue or MR, and there are newly-mentioned users to email, we
still use the :new_issue and :new_merge_request actions, so this works for that
case as well.
Diffstat (limited to 'changelogs/unreleased/38862-email-notifications-not-sent-as-expected.yml')
-rw-r--r-- | changelogs/unreleased/38862-email-notifications-not-sent-as-expected.yml | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/changelogs/unreleased/38862-email-notifications-not-sent-as-expected.yml b/changelogs/unreleased/38862-email-notifications-not-sent-as-expected.yml new file mode 100644 index 00000000000..6b1b309ab14 --- /dev/null +++ b/changelogs/unreleased/38862-email-notifications-not-sent-as-expected.yml @@ -0,0 +1,6 @@ +--- +title: Fix sending notification emails to users with the mention level set who were + mentioned in an issue or merge request description +merge_request: +author: +type: fixed |