[openstreetmap/openstreetmap-website] Adds note versions and variable note tags (PR #5904)

Nenad Vujicic notifications at github.com
Wed May 14 22:08:25 UTC 2025


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

> **Mutable text/coordinates of Notes however looks highly problematic to me.**
> 
> ...
> 
> Changing the text or coordinates of an original note (or previous comment) at a later time, (potentially) changes the meaning of all comments that were written between time where original text was written and its updated version. Also, some apps cache notes offline, which would further make determining on which version the reply was made problematic.

Yes, I agree with you. I'll probably (if others agree and they probably will :shrug:) leave only updating note tags in PR (and status, of course, through closing / reopening / hiding). Btw, I haven't planned to enable editing comments :satisfied:

> Also, one should consider that there will be both old API apps and new API apps running at the same time for forseeable future, so they would need to work together without stepping on each other toes.

I haven't planned to change API, but only to extend it.

> Also, how would the [Notes planet dump](https://planet.openstreetmap.org/notes/) change with mutable notes? It seems like a drastic change, perhaps even necessitating splitting to separate planet-notes and history-notes as we have for elements (compared to e.g. mere adding of note tags, which would probably just add "tags=" attribute to appropriate XML element, which would work as-is in most if not all data consumers, and is trivial change)

I think this is very good question even if we allow changing only note-tags, thanks!

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

Message ID: <openstreetmap/openstreetmap-website/pull/5904/c2881697521 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250514/08560c53/attachment.htm>


More information about the rails-dev mailing list