<p dir="auto">Instead of settings named like this <code class="notranslate">Settings.max_changeset_query_limit</code><br>
it will be easier to work with <code class="notranslate">Settings.changesets.max_query_limit</code>.</p>
<p dir="auto">It matches the api capabilities response structure better.<br>
It will be easier to create a generic <a href="https://github.com/openstreetmap/openstreetmap-website/blob/406d90c5c8da5633324bb1d28d7a2103a61c95e4/app/controllers/api/changesets_controller.rb#L426"><code class="notranslate">result_limit</code> method</a>.</p>
<p dir="auto">Also added settings validation in <code class="notranslate">config/initializers/config.rb</code>.</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/4189'>https://github.com/openstreetmap/openstreetmap-website/pull/4189</a></p>
<h4>Commit Summary</h4>
<ul>
<li><a href="https://github.com/openstreetmap/openstreetmap-website/pull/4189/commits/af68b361046a0d9d68acfdc1db72ae02deadfa58" class="commit-link">af68b36</a> Change query limit settings format to nested mappings</li>
</ul>
<h4 style="display: inline-block">File Changes </h4> <p style="display: inline-block">(<a href="https://github.com/openstreetmap/openstreetmap-website/pull/4189/files">9 files</a>)</p>
<ul>
<li>
<strong>M</strong>
<a href="https://github.com/openstreetmap/openstreetmap-website/pull/4189/files#diff-6726f8601cd22886963fb41888b55956cae4f8124aa12c701da77c3add44d94c">app/controllers/api/changesets_controller.rb</a>
(6)
</li>
<li>
<strong>M</strong>
<a href="https://github.com/openstreetmap/openstreetmap-website/pull/4189/files#diff-6e9fe051c0ef757618e376ffe731dfca8d4960ef0838b7d6a05609b46ba0fad1">app/controllers/api/notes_controller.rb</a>
(6)
</li>
<li>
<strong>M</strong>
<a href="https://github.com/openstreetmap/openstreetmap-website/pull/4189/files#diff-5346abcc9a6688e08e413aa0cb0a0ae68549121499776c77027b985a4151c5df">app/views/api/capabilities/show.builder</a>
(8)
</li>
<li>
<strong>M</strong>
<a href="https://github.com/openstreetmap/openstreetmap-website/pull/4189/files#diff-08932fbf37b82da16f8800a3350d03ef942c041ecd788dfe1e060e67516291a3">app/views/api/capabilities/show.json.jbuilder</a>
(8)
</li>
<li>
<strong>M</strong>
<a href="https://github.com/openstreetmap/openstreetmap-website/pull/4189/files#diff-8d22082941b3ca9f588bb2b795ae8f86f233c0175d799a8703dad41458087832">config/initializers/config.rb</a>
(8)
</li>
<li>
<strong>M</strong>
<a href="https://github.com/openstreetmap/openstreetmap-website/pull/4189/files#diff-e769bbb8c1ba3711c5403b424ed9c218ffafba7f1890ee394717196f28ff4540">config/settings.yml</a>
(18)
</li>
<li>
<strong>M</strong>
<a href="https://github.com/openstreetmap/openstreetmap-website/pull/4189/files#diff-6bba4618419284e74945f52232db102b27450b8bf3a6289671a5dbb1eb20d8b5">test/controllers/api/capabilities_controller_test.rb</a>
(16)
</li>
<li>
<strong>M</strong>
<a href="https://github.com/openstreetmap/openstreetmap-website/pull/4189/files#diff-e95198c7f7744be3ce9857db36d2374538ca7d64caaf0f6f85865d6752563ee9">test/controllers/api/changesets_controller_test.rb</a>
(15)
</li>
<li>
<strong>M</strong>
<a href="https://github.com/openstreetmap/openstreetmap-website/pull/4189/files#diff-b4cc80ae5a4279eef2a7494cb8c489220ce03acaf36932b2042247df9d316908">test/controllers/api/notes_controller_test.rb</a>
(20)
</li>
</ul>
<h4>Patch Links:</h4>
<ul>
<li><a href='https://github.com/openstreetmap/openstreetmap-website/pull/4189.patch'>https://github.com/openstreetmap/openstreetmap-website/pull/4189.patch</a></li>
<li><a href='https://github.com/openstreetmap/openstreetmap-website/pull/4189.diff'>https://github.com/openstreetmap/openstreetmap-website/pull/4189.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/4189">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLN2WGMDUQOGJ7E2FJ3XWDR5TANCNFSM6AAAAAA3WVMD6M">unsubscribe</a>.<br />You are receiving this because you are subscribed to this thread.<img src="https://github.com/notifications/beacon/AAK2OLOI4OONYVBX5SFVB2LXWDR5TA5CNFSM6AAAAAA3WVMD6OWGG33NNVSW45C7OR4XAZNFJFZXG5LFVJRW63LNMVXHIX3JMTHG5O47MA.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/4189</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/4189",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/4189",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>