<p dir="auto">Expired/revoked blocks have no edit links. Editing them results in "The block has already expired and cannot be edited."</p>
<p dir="auto">This PR removes the <code class="notranslate">active?</code> check on update actions which allows editing inactive blocks.</p>
<h2 dir="auto">Why?</h2>
<p dir="auto">Some block are issued automatically and there are false positives among them. Any moderator can revoke erroneous blocks but it's also desirable to edit the block messages. This may be impossible for two reasons:</p>
<ol dir="auto">
<li>Revoked block is already expired.</li>
<li>Revoked block was created by a different moderator.</li>
</ol>
<p dir="auto">This PR removes reason 1.</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/5014'>https://github.com/openstreetmap/openstreetmap-website/pull/5014</a></p>

<h4>Commit Summary</h4>
<ul>
  <li><a href="https://github.com/openstreetmap/openstreetmap-website/pull/5014/commits/51708d6238b933f6ca96fe05fcab2e5437cd3d3b" class="commit-link">51708d6</a>  Allow to edit inactive blocks</li>
  <li><a href="https://github.com/openstreetmap/openstreetmap-website/pull/5014/commits/d750cc3e808191e9a5a7a43148bd91b42a1b16e8" class="commit-link">d750cc3</a>  Show edit links for inactive blocks in block lists</li>
  <li><a href="https://github.com/openstreetmap/openstreetmap-website/pull/5014/commits/6ec1947e905bf38593bf9202855bc05fda45f808" class="commit-link">6ec1947</a>  Show edit links for inactive blocks on block pages</li>
</ul>

<h4 style="display: inline-block">File Changes </h4> <p style="display: inline-block">(<a href="https://github.com/openstreetmap/openstreetmap-website/pull/5014/files">5 files</a>)</p>
<ul>
  <li>
    <strong>M</strong>
    <a href="https://github.com/openstreetmap/openstreetmap-website/pull/5014/files#diff-5fb62154bcc2028d8f094225ee59e06e6dc57b853e0d429f7ebab0168f428111">app/controllers/user_blocks_controller.rb</a>
    (3)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/openstreetmap/openstreetmap-website/pull/5014/files#diff-5600ede6a40cd2ed0b592ddd5fd56e0a863da7c72dc1e80443d3ebd056b9a7e5">app/views/user_blocks/_block.html.erb</a>
    (2)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/openstreetmap/openstreetmap-website/pull/5014/files#diff-d669c04f0a7e45151d990c7457c79db874d8d6c47e41ee6817048eeb600cc9a1">app/views/user_blocks/show.html.erb</a>
    (5)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/openstreetmap/openstreetmap-website/pull/5014/files#diff-44438ce218f5287c58d0017f965d888715635d94280669896f75841fbd7b4cd7">config/locales/en.yml</a>
    (1)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/openstreetmap/openstreetmap-website/pull/5014/files#diff-9ca0f3412d30fa75d21d710e7d0f07b08a8eeb934543b9db405b8d63619548c1">test/controllers/user_blocks_controller_test.rb</a>
    (96)
  </li>
</ul>

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