[openstreetmap/openstreetmap-website] Add UserMute to control message visibility (PR #4284)
Gregory Igelmund
notifications at github.com
Wed Dec 6 22:06:41 UTC 2023
@grekko commented on this pull request.
> @@ -33,6 +33,6 @@ mail = Mail.new($stdin.read
message = Message.from_mail(mail, from, to)
message.save!
-UserMailer.message_notification(message).deliver
+UserMailer.message_notification(message).deliver_later if message.notify_recipient?
```suggestion
UserMailer.message_notification(message).deliver if message.notify_recipient?
```
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/4284#pullrequestreview-1768707329
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/4284/review/1768707329 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20231206/952adfa4/attachment.htm>
More information about the rails-dev
mailing list