[openstreetmap/openstreetmap-website] Adds optional use of notes records (PR #5511)
Nenad Vujicic
notifications at github.com
Tue Jan 28 17:59:25 UTC 2025
> The commits here are doing things in a strange order. One way to untangle them is to make another pull request that drops author_id/_ip without doing other things.
Added #5568
The problem with the current solution is JS is breaking (displaying only a subset of notes from bounding box) when trying to display note without visible comments (e.g. we create new user, login as that user, create new note without further commenting / making other actions, delete user, navigate to area where previously created note is located). This bug is solved by #3617 but it's still not merged. Shall we wait #3617 to be merged or add its solution to this PR or deal with these problematic notes in some other way?
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5511#issuecomment-2619703679
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/5511/c2619703679 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250128/96db5589/attachment-0001.htm>
More information about the rails-dev
mailing list