[openstreetmap/openstreetmap-website] Allow users to hide the heatmap (PR #6606)

[Tom Hughes]

@tomhughes commented on this pull request.



> @@ -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?)

So we decided against always showing it to moderators and/or administrators?

In fact as it stands it isn't even shown to the user themselves?

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6606#pullrequestreview-3594026068
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/pull/6606/review/3594026068 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20251218/6f3f7d37/attachment.htm>