<div style="display: flex; flex-wrap: wrap; white-space: pre-wrap; align-items: center; "><img height="20" width="20" style="border-radius:50%; margin-right: 4px;" decoding="async" src="https://avatars.githubusercontent.com/u/156656?s=20&v=4" /><strong>mnalis</strong> left a comment <a href="https://github.com/openstreetmap/openstreetmap-website/pull/5904#issuecomment-2858692186">(openstreetmap/openstreetmap-website#5904)</a></div>
<p dir="auto">I think this PR would benefit from outlying in more details exactly <em><strong>why</strong></em> is it doing changes (i.e. what exactly new user-requested functionality will become available), instead of just enumerating low-level technical steps which are being done. Judging by few words and  linked PR discussion, I guess it might have something to do with enabling users to add tags to notes and thus making notes mutable, but I cannot really tell what is an actual idea being implemented, and what are technical side-effects.</p>
<blockquote>
<p dir="auto">plus update functionality (we would have new event type - update) something you would like?</p>
</blockquote>
<p dir="auto">I don't think that notes would benefit from being changed, in fact it would likely be very bad. I.e. if note could be moved or its description changed, followup comments could be misunderstood or turned out of context. Also, it would make necessery not only API/UI to see what changes were being made and when, but a new (or extended) planet.osm.org dump (like we have e.g. <code class="notranslate">planet-*.bz2</code> as well as <code class="notranslate">history-*.bz2</code> for nodes/ways/relations).</p>
<p dir="auto">Only part of Note that should be modifiable IMHO is implementing #hashtags, the rest of the notes should better remain immutable. Having versions, locations and descriptions being change are IMHO likely to introduce much more chaos then help.</p>
<p dir="auto"><strong>TL;DR:</strong> can you list example use-cases and how this PR addresses them (i.e. <em>"users currently do this thing xxxxx which is inconvenient, and with those changes implemented they will be able do this other thing which is easier/better because yyyy"</em>)</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />Reply to this email directly, <a href="https://github.com/openstreetmap/openstreetmap-website/pull/5904#issuecomment-2858692186">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLIDRWPXDUO5OUAZMDL25IFY5AVCNFSM6AAAAAB2TRU5CGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDQNJYGY4TEMJYGY">unsubscribe</a>.<br />You are receiving this because you are subscribed to this thread.<img src="https://github.com/notifications/beacon/AAK2OLPTTBPVLIOCJB2VDFT25IFY5A5CNFSM6AAAAAB2TRU5CGWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTVKMQXFU.gif" height="1" width="1" alt="" /><span style="color: transparent; font-size: 0; display: none; visibility: hidden; overflow: hidden; opacity: 0; width: 0; height: 0; max-width: 0; max-height: 0; mso-hide: all">Message ID: <span><openstreetmap/openstreetmap-website/pull/5904/c2858692186</span><span>@</span><span>github</span><span>.</span><span>com></span></span></p>

<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/openstreetmap/openstreetmap-website/pull/5904#issuecomment-2858692186",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/5904#issuecomment-2858692186",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>