<p></p>
<p dir="auto">Updated reasoning about entry points:</p>
<ul dir="auto">
<li><code class="notranslate">/api/0.6/notes</code><br>
There are proposals to show notes on the website by default. One objection to that is performance. Notes for the note layer are fetched with this call, and it's not implemented in cgimap. Adding more parameters to it makes a potential fast implementation more complicated.</li>
<li>My original pull request was intended to be similar to element multifetch calls. That would also be a reason to put it at <code class="notranslate">/api/0.6/notes</code>, whoch has its drawbacks. But it may also be undesirable for another reason. Element multifetches fail if there is even a single element that can't be shown. You may request hundreds of elements and an error response. Theres's no partial result in this response and no indication which element caused the error. Meanwhile changeset query by ids just skips changesets it can't show.</li>
<li><code class="notranslate">/api/0.6/notes/search</code><br>
This seems like a better option in case we don't want to introduce another entry point and if we want feature parity with changeset searching. But there's a drawback. If you supply the note ids as the only parameter, you won't necessarily receive all of the requested notes even if they exist and not hidden. By default the close date filter is enabled, which will skip notes closed more than a week ago. Actual multifetch-like call should include <code class="notranslate">closed=-1</code> parameter to counteract that.</li>
</ul>

<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/3707#issuecomment-1676065095">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLL7RHOTYZAI7OFEZI3XU7J25ANCNFSM6AAAAAAQSADJUU">unsubscribe</a>.<br />You are receiving this because you are subscribed to this thread.<img src="https://github.com/notifications/beacon/AAK2OLOZYWLBFPN5WCM7XGLXU7J25A5CNFSM6AAAAAAQSADJUWWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTTD424UO.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/3707/c1676065095</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/3707#issuecomment-1676065095",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/3707#issuecomment-1676065095",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>