<p dir="auto">Currently blocks on users act as api write access blocks. Users can still do any modifications not involving the api. Some modifications can be done both through the api and through the web interface. Uploading and editing traces is one of them.</p>
<p dir="auto">As a result it's impossible to stop users from uploading questionable traces. And if we want to stop them, why should it matter if their uploads are coming through the api or not? Maybe sometimes it matters, for example, if they're using a faulty app, but that's secondary to their ability to upload at all. Although maybe we should let them delete their traces even when blocked.</p>
<p dir="auto">In this PR users will get this flash message if they try to write or edit traces:</p>
<p dir="auto"><a target="_blank" rel="noopener noreferrer" href="https://user-images.githubusercontent.com/4158490/257589258-9ea8ba7d-6414-42bd-8822-d556e5a11620.png"><img src="https://user-images.githubusercontent.com/4158490/257589258-9ea8ba7d-6414-42bd-8822-d556e5a11620.png" alt="trace" style="max-width: 100%;"></a></p>
<p dir="auto">The button links to the block page.</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/4129'>https://github.com/openstreetmap/openstreetmap-website/pull/4129</a></p>
<h4>Commit Summary</h4>
<ul>
<li><a href="https://github.com/openstreetmap/openstreetmap-website/pull/4129/commits/a1d4f6d2298ff472138eb6f62adf7f0830754496" class="commit-link">a1d4f6d</a> Remove ability to write traces from blocked users</li>
<li><a href="https://github.com/openstreetmap/openstreetmap-website/pull/4129/commits/8aefa6c369d81c690ebef6d204a51cc16fd0dfc2" class="commit-link">8aefa6c</a> Show link to block in blocked trace flash message</li>
<li><a href="https://github.com/openstreetmap/openstreetmap-website/pull/4129/commits/0198645e907bc104cbfb65bcb977f51f6ae95c8c" class="commit-link">0198645</a> Style link in flash block message as a button</li>
<li><a href="https://github.com/openstreetmap/openstreetmap-website/pull/4129/commits/b775b0af1e98bf74fc433ba1221675829e5557fe" class="commit-link">b775b0a</a> Make different blocked warning/redirect for trace modification</li>
<li><a href="https://github.com/openstreetmap/openstreetmap-website/pull/4129/commits/97b671214c7f2fce81600e21b97e441d4585dce0" class="commit-link">97b6712</a> Move trace-specific deny access code to trace controller</li>
<li><a href="https://github.com/openstreetmap/openstreetmap-website/pull/4129/commits/2b9a764eefecc49934b6380ca695cfd8f5e0e4cc" class="commit-link">2b9a764</a> Define specific current user access denial only for html requests</li>
<li><a href="https://github.com/openstreetmap/openstreetmap-website/pull/4129/commits/c0e4a51fbe293a492e0bbeaadb9f9885d6c745b3" class="commit-link">c0e4a51</a> Add trace ability tests</li>
</ul>
<h4 style="display: inline-block">File Changes </h4> <p style="display: inline-block">(<a href="https://github.com/openstreetmap/openstreetmap-website/pull/4129/files">6 files</a>)</p>
<ul>
<li>
<strong>M</strong>
<a href="https://github.com/openstreetmap/openstreetmap-website/pull/4129/files#diff-debf59904131dc89a88ab44cbb1d6deaaa5a7d70c0507f674080df62f7c8e41f">app/abilities/ability.rb</a>
(4)
</li>
<li>
<strong>M</strong>
<a href="https://github.com/openstreetmap/openstreetmap-website/pull/4129/files#diff-766c34fd6533171eaf54300c153f89d6002c35c02cfc9c5b219251f85180ad07">app/controllers/application_controller.rb</a>
(8)
</li>
<li>
<strong>M</strong>
<a href="https://github.com/openstreetmap/openstreetmap-website/pull/4129/files#diff-2602c06a2618f83c1470c924563b55c738e3c94ffb6480e637ca0f4ce130eaba">app/controllers/traces_controller.rb</a>
(19)
</li>
<li>
<strong>A</strong>
<a href="https://github.com/openstreetmap/openstreetmap-website/pull/4129/files#diff-97650becb8684c8e44c0df39194b19e67566fdbe3c1a13f771eccf53b9187de0">app/views/traces/_blocked_flash.html.erb</a>
(4)
</li>
<li>
<strong>M</strong>
<a href="https://github.com/openstreetmap/openstreetmap-website/pull/4129/files#diff-44438ce218f5287c58d0017f965d888715635d94280669896f75841fbd7b4cd7">config/locales/en.yml</a>
(4)
</li>
<li>
<strong>M</strong>
<a href="https://github.com/openstreetmap/openstreetmap-website/pull/4129/files#diff-66914b664fcadd6ba28bd16a44eb853ca3fd4e1f9ea1e4be390f12aeea2a0697">test/abilities/abilities_test.rb</a>
(24)
</li>
</ul>
<h4>Patch Links:</h4>
<ul>
<li><a href='https://github.com/openstreetmap/openstreetmap-website/pull/4129.patch'>https://github.com/openstreetmap/openstreetmap-website/pull/4129.patch</a></li>
<li><a href='https://github.com/openstreetmap/openstreetmap-website/pull/4129.diff'>https://github.com/openstreetmap/openstreetmap-website/pull/4129.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/4129">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLPYRUKINDEIEIQGFNLXTE255ANCNFSM6AAAAAA3ACSHSI">unsubscribe</a>.<br />You are receiving this because you are subscribed to this thread.<img src="https://github.com/notifications/beacon/AAK2OLKWS533P355PRQMLZTXTE255A5CNFSM6AAAAAA3ACSHSKWGG33NNVSW45C7OR4XAZNFJFZXG5LFVJRW63LNMVXHIX3JMTHG2LOLHQ.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/4129</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/4129",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/4129",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>