diff --git a/app/models/notifier/card_event_notifier.rb b/app/models/notifier/card_event_notifier.rb index f36a1f12c..15768ebc3 100644 --- a/app/models/notifier/card_event_notifier.rb +++ b/app/models/notifier/card_event_notifier.rb @@ -6,7 +6,7 @@ class Notifier::CardEventNotifier < Notifier def recipients case source.action when "card_assigned" - source.assignees.excluding(creator, *source.collection.access_only_users) + source.assignees.excluding(creator) when "card_published" watchers_and_subscribers(include_only_watching: true).without(creator, *card.mentionees) when "comment_created" diff --git a/test/models/notifier/event_notifier_test.rb b/test/models/notifier/event_notifier_test.rb index 395d5bfa1..6429874e5 100644 --- a/test/models/notifier/event_notifier_test.rb +++ b/test/models/notifier/event_notifier_test.rb @@ -87,4 +87,12 @@ class Notifier::EventNotifierTest < ActiveSupport::TestCase Notifier.for(events(:layout_commented)).notify end end + + test "assignment events notify assignees regardless of involvement level" do + collections(:writebook).access_for(users(:jz)).access_only! + + notifications = Notifier.for(events(:logo_assignment_jz)).notify + + assert_equal [ users(:jz) ], notifications.map(&:user) + end end