[openstreetmap/openstreetmap-website] Allow users to change trace visibility in bulk (PR #7324)

Ruben L. Mendoza notifications at github.com
Mon Aug 24 21:04:55 UTC 2026


@Rub21 commented on this pull request.



> @@ -11,6 +11,11 @@
           <%= link_to t(".remove_tag_filter", :tag => params[:tag]), { :controller => "traces", :action => "index", :tag => nil } %>
         </li>
       <% end %>
+      <% if current_user && current_user == @target_user %>
+        <li>
+          <%= link_to t(".change_visibility"), change_visibility_traces_path %>
+        </li>
+      <% end %>

> Additionally, "Change Visibility" may lead to users thinking that this is a general feature, rather than a "temporary" one with a very specific aim. Should we instead say something like "Migrate legacy traces", "Change visibility of legacy traces", or something like that?

I think we can add a short explanation on the page saying that this is a temporary feature, and also say for how long we plan to keep it. but thre is no time limit for that  to remove legacy tracks.

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

Message ID: <openstreetmap/openstreetmap-website/pull/7324/review/5012544539 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20260824/25edc9bc/attachment.htm>


More information about the rails-dev mailing list