[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