[openstreetmap/openstreetmap-website] Adds version to notes table and creates note_versions table (PR #5815)

Nenad Vujicic notifications at github.com
Thu May 1 20:00:32 UTC 2025


nenad-vujicic left a comment (openstreetmap/openstreetmap-website#5815)

@tomhughes @AntonKhorev @gravitystorm 

Do we have some conclusions about notes versioning / tags? At the beginning I proposed non-mutable note tags in #5344, then you requested notes versioning ([here](https://github.com/openstreetmap/openstreetmap-website/issues/5294#issuecomment-2538781672) and [here](https://github.com/openstreetmap/openstreetmap-website/issues/5294#issuecomment-2541604426)) what is implemented in #5904 (also removing event column and non-present note-comments) and this PR is the first part of all of this.

If you don't like this (because of complexity or something else), we can do #5344 with update functionality (which will allow changing tags (or entire note?)), but I'm not sure allowing permanently deletions is good thing. Also, I'm not sure how we would do removing non-present note-comments without breaking backward compatibility.

I would like to make some contribution in area of notes, but I cannot do it without your convergence on these topics, so I would highly appreciate if we could make some move forward about this.

Thanks,
Nenad.

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

Message ID: <openstreetmap/openstreetmap-website/pull/5815/c2845641775 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250501/bc651959/attachment-0001.htm>


More information about the rails-dev mailing list