[openstreetmap/openstreetmap-website] Fixed notifications reload error (PR #7359)

Mohd Farhan Akhtar notifications at github.com
Thu Sep 3 15:41:38 UTC 2026


imfarhan0105-bits left a comment (openstreetmap/openstreetmap-website#7359)

> Well this is the fundamental question - what is the correct fix here?
> 
> Is the correct fix to remove the notification? To keep it but to render it specially without any details? Something else?

Personally, I feel like removing the notification is the better way to handle it. The notification is tied to the actual Follow/Trace record, so once that record is gone, I don't think there's much point in keeping the notification around either.

For example, if someone unfollows a user, having a “new follower” notification still sitting there afterwards feels a bit weird, since that relationship doesn't exist anymore.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/7359#issuecomment-5528203496
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/pull/7359/c5528203496 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20260903/5bda82b0/attachment-0001.htm>


More information about the rails-dev mailing list