> 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/c2619703...@github.com>
_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev

Reply via email to