[openstreetmap/openstreetmap-website] Configure queues for new notification system (PR #6929)
Andy Allan
notifications at github.com
Tue Mar 24 09:51:22 UTC 2026
gravitystorm left a comment (openstreetmap/openstreetmap-website#6929)
> While doing this I also confirmed that using `deliver_later` on a notification is pointless as it's actually just an alias for `deliver` so I've changed that as well.
We came across that during review (see https://github.com/openstreetmap/openstreetmap-website/pull/6837#discussion_r2917605521 ) and we figured that it was clearer to use `deliver_later` - that indicates to anyone reading the code that the notification is handled asynchronously, whereas the `deliver` alias might imply that the notifications are calculated synchronously.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6929#issuecomment-4116811301
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/6929/c4116811301 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20260324/64ac85cd/attachment.htm>
More information about the rails-dev
mailing list