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

[Ruben L. Mendoza]

@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 %>

> Also it's a non-standard location, next to the descriptor of the current tab. At the very least, I think it should be a new tab.

Good point. I i think we would call "Legacy Traces", and then use a clearer title on the page itself, for example "Change the visibility of your legacy traces", with a short line below explaining that these are traces that still use the old private and public values.

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

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