<p dir="auto">This is an alternative to <a class="issue-link js-issue-link" data-error-text="Failed to load title" data-id="1309642718" data-permission-text="Title is private" data-url="https://github.com/openstreetmap/openstreetmap-website/issues/3605" data-hovercard-type="pull_request" data-hovercard-url="/openstreetmap/openstreetmap-website/pull/3605/hovercard" href="https://github.com/openstreetmap/openstreetmap-website/pull/3605">#3605</a>.</p>
<p dir="auto"><a class="issue-link js-issue-link" data-error-text="Failed to load title" data-id="1309642718" data-permission-text="Title is private" data-url="https://github.com/openstreetmap/openstreetmap-website/issues/3605" data-hovercard-type="pull_request" data-hovercard-url="/openstreetmap/openstreetmap-website/pull/3605/hovercard" href="https://github.com/openstreetmap/openstreetmap-website/pull/3605">#3605</a> adds the time <code class="notranslate">from</code>/<code class="notranslate">to</code> parameters to the index endpoint that already supports the <code class="notranslate">bbox</code>, this PR adds <code class="notranslate">bbox</code> to the search that already supports <code class="notranslate">from</code>/<code class="notranslate">to</code>.</p>
<p dir="auto">You may want this PR if:</p>
<ul dir="auto">
<li>The index endpoint is to be kept as simple as possible. The index endpoint is probably used more frequently because of the notes layer. Maybe you want to optimize it further, for example, by implementing it in cgimap. In this case the fewer parameters it has, the better.</li>
<li>Changesets search already supports <code class="notranslate">bbox</code>.</li>
</ul>
<p dir="auto">For the use case described in <a class="issue-link js-issue-link" data-error-text="Failed to load title" data-id="1309642718" data-permission-text="Title is private" data-url="https://github.com/openstreetmap/openstreetmap-website/issues/3605" data-hovercard-type="pull_request" data-hovercard-url="/openstreetmap/openstreetmap-website/pull/3605/hovercard" href="https://github.com/openstreetmap/openstreetmap-website/pull/3605">#3605</a> you can switch over from the index endpoint to search if you want to download more notes.</p>

<hr>

<h4>You can view, comment on, or merge this pull request online at:</h4>
<p>  <a href='https://github.com/openstreetmap/openstreetmap-website/pull/4222'>https://github.com/openstreetmap/openstreetmap-website/pull/4222</a></p>

<h4>Commit Summary</h4>
<ul>
  <li><a href="https://github.com/openstreetmap/openstreetmap-website/pull/4222/commits/b6b4d6d472652c56f42ba4b0e9f4f526e99a00a6" class="commit-link">b6b4d6d</a>  Add bbox parameter to notes search api</li>
</ul>

<h4 style="display: inline-block">File Changes </h4> <p style="display: inline-block">(<a href="https://github.com/openstreetmap/openstreetmap-website/pull/4222/files">2 files</a>)</p>
<ul>
  <li>
    <strong>M</strong>
    <a href="https://github.com/openstreetmap/openstreetmap-website/pull/4222/files#diff-6e9fe051c0ef757618e376ffe731dfca8d4960ef0838b7d6a05609b46ba0fad1">app/controllers/api/notes_controller.rb</a>
    (37)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/openstreetmap/openstreetmap-website/pull/4222/files#diff-b4cc80ae5a4279eef2a7494cb8c489220ce03acaf36932b2042247df9d316908">test/controllers/api/notes_controller_test.rb</a>
    (31)
  </li>
</ul>

<h4>Patch Links:</h4>
<ul>
  <li><a href='https://github.com/openstreetmap/openstreetmap-website/pull/4222.patch'>https://github.com/openstreetmap/openstreetmap-website/pull/4222.patch</a></li>
  <li><a href='https://github.com/openstreetmap/openstreetmap-website/pull/4222.diff'>https://github.com/openstreetmap/openstreetmap-website/pull/4222.diff</a></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/4222">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLLMBKMZ6IV2IZGFMD3XYQ2XPANCNFSM6AAAAAA4JEUKGA">unsubscribe</a>.<br />You are receiving this because you are subscribed to this thread.<img src="https://github.com/notifications/beacon/AAK2OLPQBNTVQ3XPFHQOEQLXYQ2XPA5CNFSM6AAAAAA4JEUKGCWGG33NNVSW45C7OR4XAZNFJFZXG5LFVJRW63LNMVXHIX3JMTHG77YGN4.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/4222</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/4222",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/4222",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>