This repository has been archived on 2023-02-01. You can view files and clone it, but cannot push or open issues or pull requests.

zeripath 2e36ba0a00 Send mail to issue/pr assignee/reviewer also when OnMention is set (#18707) (#18765) ... Backport #18707 Addresses #17892, where emails notifications are not sent to assignees (issue and PR) and reviewers (PR) when they have the email setting Only email on mention enabled. From the user experience perspective, when a user gets a issue/PR assigned or a PR review request, he/she would expect to be implicitly mentioned since the assignment or request is personal and targeting a single person only. Thus I see #17892 as a bug. Could we therefore mark this ticket as such? The changed code just explicitly checks for the EmailNotificationsOnMention setting beside the existing EmailNotificationsEnabled check. Too rude? @lunny mentioned a mock mail server for tests, is there something ready. How could I make use of it? See #12774 (comment) Fix #17892 Co-authored-by: flozzone <flozzone@gmail.com>		2022-02-14 21:13:41 +08:00
..
action	Simplify parameter types (#18006)	2021-12-20 04:41:31 +00:00
base	Simplify parameter types (#18006)	2021-12-20 04:41:31 +00:00
indexer	Simplify parameter types (#18006)	2021-12-20 04:41:31 +00:00
mail	Send mail to issue/pr assignee/reviewer also when OnMention is set (#18707) (#18765)	2022-02-14 21:13:41 +08:00
ui	Some repository refactors (#17950)	2021-12-12 23:48:20 +08:00
webhook	Simplify parameter types (#18006)	2021-12-20 04:41:31 +00:00
notification.go	Simplify parameter types (#18006)	2021-12-20 04:41:31 +00:00