<p></p>
<blockquote>
<p dir="auto">So it seems that restriction was relaxed in postgres 12 which is why the main tests work as the GitHub ubuntu images use postgres 14 but our docker setup still uses postgres 11...</p>
</blockquote>
<p dir="auto">Great, thanks for looking into that. I'm happy to bump the requirement to postgres 12+ (since that's the lowest version available in the oldest Ubuntu LTS, 20.04) so it shouldn't affect too many developers/deployments (hopefully!). Should we use 12 in the docker tests to pick anything up that might affect focal environments? I'm happy either way.</p>
<p dir="auto">Only remaining thing after that is to update INSTALL.md with the new minimum version.</p>
<p dir="auto">I like that the algorithm is based on the one that we introduced for changeset comments recently, so we might be able to refactor that in future (perhaps initially moving all the user rate limits into a concern, to avoid giant <code class="notranslate">user.rb</code>). I'm also happy with the concept of the <code class="notranslate">importer</code> flag, in order to provide a pragmatic compromise for these accounts (and everyone already follows the Import Guidelines and uses a separate account, right? 😄 ).</p>
<p dir="auto">I considered suggesting reworking the tests slightly (to use <code class="notranslate">Numeric::divmod</code> and <code class="notranslate">create_list</code>) but I don't think it's worth the additional refactoring at this point.</p>
<p dir="auto">Otherwise I'm happy with this PR, and unless you have any more changes planned then I think it can be merged.</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/4319#issuecomment-1787503462">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLLPBE45S3S26VVFT4TYCENMNAVCNFSM6AAAAAA6VBWHDWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBXGUYDGNBWGI">unsubscribe</a>.<br />You are receiving this because you are subscribed to this thread.<img src="https://github.com/notifications/beacon/AAK2OLNWQJ5K5WWPDEAXZUDYCENMNA5CNFSM6AAAAAA6VBWHDWWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTTKRMRWM.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/4319/c1787503462</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/4319#issuecomment-1787503462",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/4319#issuecomment-1787503462",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>