<p dir="auto">This is a follow on to <a class="issue-link js-issue-link" data-error-text="Failed to load title" data-id="2832582904" data-permission-text="Title is private" data-url="https://github.com/openstreetmap/openstreetmap-website/issues/5609" data-hovercard-type="pull_request" data-hovercard-url="/openstreetmap/openstreetmap-website/pull/5609/hovercard" href="https://github.com/openstreetmap/openstreetmap-website/pull/5609">#5609</a> that does a couple of things.</p>
<p dir="auto">Firstly it adds a test for a note whose original author has been deleted.</p>
<p dir="auto">Secondly it corrects the test for whether to drop the first comment to align with what the note model does in the <code class="notranslate">comments</code> association so that the first comment is only dropped if the author was active because in other cases the model will already have excluded it. This primarily affects users in state <code class="notranslate">suspended</code> I think.</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/5627'>https://github.com/openstreetmap/openstreetmap-website/pull/5627</a></p>

<h4>Commit Summary</h4>
<ul>
  <li><a href="https://github.com/openstreetmap/openstreetmap-website/pull/5627/commits/49dfae87ab1913b161aaa9455a9ac89a091faed3" class="commit-link">49dfae8</a>  Add a test for viewing a note opened by a deleted user</li>
  <li><a href="https://github.com/openstreetmap/openstreetmap-website/pull/5627/commits/cb51f46a4ff913985d535afed0fd8d7d1d955126" class="commit-link">cb51f46</a>  Correct test for notes opened by inactive users</li>
</ul>

<h4 style="display: inline-block">File Changes </h4> <p style="display: inline-block">(<a href="https://github.com/openstreetmap/openstreetmap-website/pull/5627/files">2 files</a>)</p>
<ul>
  <li>
    <strong>M</strong>
    <a href="https://github.com/openstreetmap/openstreetmap-website/pull/5627/files#diff-c254740097a8c0601832077488db77e572b2c2b926f77508fadc6081b0052804">app/controllers/notes_controller.rb</a>
    (2)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/openstreetmap/openstreetmap-website/pull/5627/files#diff-b41a560e061b578bbf13158bf175036a9489241ab69f59137c44d57df9682fa7">test/controllers/notes_controller_test.rb</a>
    (9)
  </li>
</ul>

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