Skip to content

Commit

Permalink
Merge pull request #15276 from danidoni/fix-undefined-method-rss-secr…
Browse files Browse the repository at this point in the history
…et-for-group

Fix issue when creating notification with group and RSS
  • Loading branch information
eduardoj committed Nov 29, 2023
2 parents ba418b9 + 646f059 commit a76ac23
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/api/app/services/notification_service/notifier.rb
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ def create_notification_per_subscription(subscription, channel)
end

def create_notification?(subscriber, channel)
return false if subscriber.nil? || subscriber.away? || (channel == :rss && subscriber.rss_secret.blank?)
return false if subscriber.nil? || subscriber.away? || (channel == :rss && subscriber.try(:rss_secret).blank?)
return false unless notifiable_exists?
return false unless create_report_notification?(event: @event, subscriber: subscriber)

Expand Down

0 comments on commit a76ac23

Please sign in to comment.