@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/[email protected]>_______________________________________________
rails-dev mailing list
[email protected]
https://lists.openstreetmap.org/listinfo/rails-dev
