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