<p></p>
<p><b>@1ec5</b> commented on this pull request.</p>

<hr>

<p>In <a href="https://github.com/openstreetmap/openstreetmap-website/pull/6606#discussion_r2632163252">app/controllers/users/heatmaps_controller.rb</a>:</p>
<pre style='color:#555'>> @@ -13,7 +13,7 @@ class HeatmapsController < ApplicationController
     def show
       @user = User.find_by(:display_name => params[:user_display_name])
 
-      if @user && (@user.visible? || current_user&.administrator?)
+      if @user&.public_heatmap? && (@user.visible? || current_user&.administrator?)
</pre>
<p dir="auto">I mentioned to <a class="user-mention notranslate" data-hovercard-type="user" data-hovercard-url="/users/pablobm/hovercard" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/pablobm">@pablobm</a> early on that we could take a conservative approach and hide it from both groups unless someone from either group said they needed it for something. (As long as it’s shown to moderators, it would only make sense for it to be shown to administrators too.)</p>
<p dir="auto">We could reasonably show the user’s own heatmap even when they opt out, but it really works against the goal of making the profile look similar to what the public sees. Unless we hide it in response to the setting, the setting will appear to have no effect unless the user opens up their profile in private browsing or something. Maybe as tail work, we could consider moving it to the dashboard in that case, so the user can at least still navigate to their own contributions by date?</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/6606#discussion_r2632163252">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLMDV7PQPWSKIZXVT4D4CLXBDAVCNFSM6AAAAACOTYJFYGVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZTKOJUGQYTQNZTG4">unsubscribe</a>.<br />You are receiving this because you are subscribed to this thread.<img src="https://github.com/notifications/beacon/AAK2OLKMAJLZWNEVGFXDSRL4CLXBDA5CNFSM6AAAAACOTYJFYGWGG33NNVSW45C7OR4XAZNRKB2WY3CSMVYXKZLTORJGK5TJMV32UY3PNVWWK3TUL5UWJTWWHZ5DC.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/6606/review/3594418737</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/6606#discussion_r2632163252",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/6606#discussion_r2632163252",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>