don't require project

it's not there in the case of personal snippets, f. ex., and we've already guarded against its being missing in #find_notification_setting

don't require project
e7d136eb · http://jneen.net/ · 8386d69f · e7d136eb
Commit e7d136eb authored Jul 31, 2017 by http://jneen.net/
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 3 deletions

notification_recipient.rb app/models/notification_recipient.rb +1 -3

No files found.
--- a/app/models/notification_recipient.rb
+++ b/app/models/notification_recipient.rb
 class NotificationRecipient
-  attr_reader :user, :project, :type
+  attr_reader :user, :type
  def initialize(user, type,
                 custom_action: nil,
                 target: nil,
@@ -13,8 +13,6 @@ class NotificationRecipient
    @project = project || @target&.project
    @user = user
    @type = type
-
-    raise ArgumentError, "Project is missing" if @project.nil?
  end

  def notification_setting