@nertc commented on this pull request.


> -                  <%= link_to t(".remove as friend"), 
> remove_friend_path(:display_name => @user.display_name), :method => :post %>
+                  <%= link_to t(".unfollow"), follow_path(:display_name => 
@user.display_name), :method => :delete %>
                 <% else %>
-                  <%= link_to t(".add as friend"), 
make_friend_path(:display_name => @user.display_name), :method => :post %>
+                  <%= link_to t(".follow"), follow_path(:display_name => 
@user.display_name), :method => :post %>

My thought was to use `:edit` as the functionality was focused on editing but 
not showing information about `follows`. But as 
https://github.com/openstreetmap/openstreetmap-website/pull/5449#issuecomment-2578415862
 suggests, if we use `:show` also as edit page where it is possible, I will 
change it accordingly. I thought your comment meant to use standard `Singular 
Resource` routes in general, not `:show` in particular.

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

Message ID: 
<openstreetmap/openstreetmap-website/pull/5261/review/2542502...@github.com>
_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev

Reply via email to