<blockquote>
<p>Agree this functionality shouldn't be to report editing errors, but there will be a need to report abusive changeset comments (eventually)</p>
</blockquote>
<p>There's definitely scope to widen the list of things that can be reported. I would certainly consider adding changeset comments, note comments and traces in followup PRs, but I'm particularly cautious about expanding the scope until we know that the workflow is OK.</p>
<blockquote>
<p>Any discussion on how to approach this yet? Or even that needs to be taken up?</p>
</blockquote>
<p>We'll need to suck-it-and-see, but we might need to add more website moderators, if the list of reports is too big for the existing group of moderators and/or administrators to handle. Currently moderator status on the website is a pretty powerful position since it includes being able to create user_blocks, which I'd prefer to tie more closely to DWG membership status. So if we find we need to add a few dozen new website moderators just to handle notes, diary_entries etc, then we'll need more fine-grained roles and permissions. Some foundational code and refactoring would be needed for that, see e.g. <a href="https://github.com/openstreetmap/openstreetmap-website/issues/1626" class="issue-link js-issue-link" data-error-text="Failed to load issue title" data-id="254033085" data-permission-text="Issue title is private" data-url="https://github.com/openstreetmap/openstreetmap-website/issues/1626">#1626</a></p>
<p>Of course, it's hard to know whether there will actually be an increase in reports being dealt with, since we currently handle most spam etc through existing out-of-band channels (e.g. using IRC, or via DWG) - this just makes the reporting in-band instead of out-of-band. Like I said, probably best to start with a small list of reportable items and see how it goes.</p>
<blockquote>
<p>What's the possible timeline for deployment of this feature?</p>
</blockquote>
<ul>
<li>Other developers now review this PR. It's sort of half-reviewed since technically the work I was doing was just improving the existing GSoC PR, with a view to taking care of things I would have asked the student to fix (if I had a time machine).</li>
<li>I can make any changes requested.</li>
<li>Gets deployed pretty quickly after that.</li>
</ul>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/openstreetmap/openstreetmap-website/pull/1576#issuecomment-369443689">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABWnLRqB7E9cJlJJQSkTyZ5UMm4kcaWrks5tZ08VgaJpZM4OOfLC">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABWnLeGIw90hSXhgmV-Gq7mqCcIyai9Dks5tZ08VgaJpZM4OOfLC.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
  <link itemprop="url" href="https://github.com/openstreetmap/openstreetmap-website/pull/1576#issuecomment-369443689"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>

<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/openstreetmap/openstreetmap-website","title":"openstreetmap/openstreetmap-website","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/openstreetmap/openstreetmap-website"}},"updates":{"snippets":[{"icon":"PERSON","message":"@gravitystorm in #1576: \u003e Agree this functionality shouldn't be to report editing errors, but there will be a need to report abusive changeset comments (eventually)\r\n\r\nThere's definitely scope to widen the list of things that can be reported. I would certainly consider adding changeset comments, note comments and traces in followup PRs, but I'm particularly cautious about expanding the scope until we know that the workflow is OK.\r\n\r\n\u003e Any discussion on how to approach this yet? Or even that needs to be taken up?\r\n\r\nWe'll need to suck-it-and-see, but we might need to add more website moderators, if the list of reports is too big for the existing group of moderators and/or administrators to handle. Currently moderator status on the website is a pretty powerful position since it includes being able to create user_blocks, which I'd prefer to tie more closely to DWG membership status. So if we find we need to add a few dozen new website moderators just to handle notes, diary_entries etc, then we'll need more fine-grained roles and permissions. Some foundational code and refactoring would be needed for that, see e.g. https://github.com/openstreetmap/openstreetmap-website/issues/1626\r\n\r\nOf course, it's hard to know whether there will actually be an increase in reports being dealt with, since we currently handle most spam etc through existing out-of-band channels (e.g. using IRC, or via DWG) - this just makes the reporting in-band instead of out-of-band. Like I said, probably best to start with a small list of reportable items and see how it goes.\r\n\r\n\u003e What's the possible timeline for deployment of this feature?\r\n\r\n* Other developers now review this PR. It's sort of half-reviewed since technically the work I was doing was just improving the existing GSoC PR, with a view to taking care of things I would have asked the student to fix (if I had a time machine).\r\n* I can make any changes requested.\r\n* Gets deployed pretty quickly after that."}],"action":{"name":"View Pull Request","url":"https://github.com/openstreetmap/openstreetmap-website/pull/1576#issuecomment-369443689"}}}</script>