[openstreetmap/openstreetmap-website] Convert more notifications to use Noticed (PR #6933)
Pablo Brasero
notifications at github.com
Wed Mar 25 10:48:23 UTC 2026
@pablobm commented on this pull request.
Just one thing: the first commit has a rogue change that should be squashed into the second commit.
On test/factories/changeset_subscriptions.rb:
Thank you, I missed this one at https://github.com/openstreetmap/openstreetmap-website/pull/6607
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6933#pullrequestreview-4005632265
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/6933/review/4005632265 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20260325/ef2d59ac/attachment.htm>
More information about the rails-dev
mailing list