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

[Pablo Brasero]

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

What @1ec5 said. I have updated the PR description to make this explicit.

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

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